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

Open
wants to merge 25 commits into
base: develop
Choose a base branch
from
Open

Api v2 tweaks #622

wants to merge 25 commits into from

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
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.

1 participant