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

chore(update_channel): better errors #771

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

lusergit
Copy link
Collaborator

When creating and updating an update channel, target group ids are validated to be available, meaning:

  • they are present in the DB
  • they are not related to any other update channel

closes #529

@coveralls
Copy link

coveralls commented Dec 10, 2024

Pull Request Test Coverage Report for Build 332fea1ed9a7cca986d54537db8ea29292b53b2c-PR-771

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+80.5%) to 80.486%

Totals Coverage Status
Change from base Build 12281651344: 80.5%
Covered Lines: 1225
Relevant Lines: 1522

💛 - Coveralls

@lusergit lusergit force-pushed the chore/errors-update-channel branch 2 times, most recently from 158b288 to 7e9c7e0 Compare December 10, 2024 16:41
@lusergit lusergit force-pushed the chore/errors-update-channel branch 2 times, most recently from aa65a2b to 0173107 Compare December 12, 2024 10:07
When creating and updating an update channel, target group ids are
validated to be available, meaning:
- they are present in the DB
- they are not related to any other update channel

closes edgehog-device-manager#529

Signed-off-by: Luca Zaninotto <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Distinguish different errors when relating an update channel to target groups
3 participants