Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Api v2 tweaks #622

Merged
merged 25 commits into from
Nov 9, 2024
Merged

Api v2 tweaks #622

merged 25 commits into from
Nov 9, 2024

Conversation

c8y3
Copy link
Contributor

@c8y3 c8y3 commented Oct 18, 2024

  • finished moving API v2 endpoints into namespace app.blueprints.rest.v2
  • polished implementation of ioc update (by the way, I avoid retrieve the ioc twice, by changing the signature of the business method iocs_update: it expects an ioc instead of an identifier, see api_v2_ioc_routes)
  • added backend tests for ioc update
  • replaced in UI request to /case/ioc/update/{identifier} by API v2
  • first UI test on the alerts page
  • fixed: when there was no relationships between users and customers, the administrator could not see any alerts
  • other code cleanup: removed some dead code, simple quotes

@c8y3 c8y3 requested a review from whikernel October 18, 2024 13:13
@c8y3 c8y3 added enhancement New feature or request quality labels Oct 18, 2024
@whikernel whikernel merged commit 8dab433 into develop Nov 9, 2024
6 checks passed
@whikernel whikernel deleted the api_v2_tweaks branch November 9, 2024 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants