[api][tools] Add swagger URL and UI for public APIs #3776
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
tabraiz12
commented
Jul 8, 2024
tabraiz12
changed the title
Harshg999
reviewed
Jul 8, 2024
Harshg999
changed the title
yes, i have tested this out. |
Harshg999
reviewed
Jul 9, 2024
Harshg999
reviewed
Jul 10, 2024
desktop/core/src/desktop/urls.py
Outdated
| @@ -202,6 +203,9 @@ | |||
| ] | |||
|
|
|||
| dynamic_patterns += [ | |||
| re_path('api/schema', SpectacularAPIView.as_view(),name='schema'), | |||
There was a problem hiding this comment.
We can remove this mapping as well, I checked and its just downloading the schema for all the APIs. Is this required?
There was a problem hiding this comment.
yes, we require this for QE.
There was a problem hiding this comment.
@tabraiz12 Can we change the mapping here to '^api/schema/?$'?
Harshg999
approved these changes
Jul 10, 2024
Change-Id: I2af75fd420ba84d6fa85a8af92013b59d60c054e
Change-Id: I080794b1d9937d8f4474d80757e0e5f299d3ea51
This reverts commit 78d90ca.
This reverts commit 8ea41bb.
This reverts commit 255e034.
Change-Id: I9be2abda3c2af2e04dd63248bb084fa45f80d4fa
Change-Id: I34e3aa43f1905ac9cf901c1ce448071c61c459d5
Change-Id: Ic1fce350d5a1e526e26e12c4b626dfd54ea352c9
Change-Id: If0d3f5744f06262a7d0723d7939670a8abdb5a41
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add Swagger UI and URL for public url for easier QE Automation
How was this patch tested?
Please review Hue Contributing Guide before opening a pull request.