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

feat(service): Include v4 APIs in Categories #7504

Merged
merged 1 commit into from
May 24, 2024

Conversation

illichso
Copy link
Contributor

@illichso illichso commented May 17, 2024

Issue

https://gravitee.atlassian.net/browse/APIM-4851

Description

Make sure we create and update apis with categories ids, not categories keys

Additional context


📚  View the storybook of this branch here

@illichso illichso requested a review from a team as a code owner May 17, 2024 13:20
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 2 times, most recently from 21d4a82 to 5aa733d Compare May 17, 2024 13:27
Copy link

cypress bot commented May 17, 2024

Passing run #1384 ↗︎

0 82 8 0 Flakiness 0

Details:

feat(service): save category id instead of key in apis table and REST responds w...
Project: APIM UI Tests Commit: dba040ef8b
Status: Passed Duration: 07:08 💡
Started: May 24, 2024 12:39 PM Ended: May 24, 2024 12:46 PM

Review all test suite changes for PR #7504 ↗︎

@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 7 times, most recently from a52dc30 to d3c002a Compare May 20, 2024 12:12
@illichso illichso requested a review from jourdiw May 20, 2024 12:13
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 6 times, most recently from 3f63734 to 057bea0 Compare May 21, 2024 07:40
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 7 times, most recently from 239c11a to 21a4695 Compare May 22, 2024 08:59
@illichso illichso requested a review from jourdiw May 22, 2024 09:55
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 2 times, most recently from 1e1f784 to a31a4cb Compare May 22, 2024 11:38
Copy link
Contributor

@jourdiw jourdiw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just make sure locally that the main use cases work with the Resources 👍

@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 6 times, most recently from d60f857 to d4d0b77 Compare May 23, 2024 14:00
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch 4 times, most recently from 78a8af0 to 8414772 Compare May 24, 2024 09:20
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch from 8414772 to 6589a77 Compare May 24, 2024 12:25
@illichso illichso force-pushed the APIM-4851-include-v4-apis-in-categories-run-e2e branch from 6589a77 to dba040e Compare May 24, 2024 12:25
@illichso illichso merged commit 5daac60 into master May 24, 2024
52 of 54 checks passed
@illichso illichso deleted the APIM-4851-include-v4-apis-in-categories-run-e2e branch May 24, 2024 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants