Skip to content

Conversation

@deborahgu
Copy link
Member

the switches to using DRF's native schema generation, because it has deprecated coreAPI:

https://www.django-rest-framework.org/community/3.10-announcement/

FIXES: APER-3932

the switches to using DRF's native schema generation, because it has
deprecated coreAPI:

https://www.django-rest-framework.org/community/3.10-announcement/

FIXES: APER-3932
@deborahgu deborahgu requested a review from a team as a code owner August 25, 2025 22:30
@github-actions
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  program_intent_engagement
  urls.py
Project Total  

This report was generated by python-coverage-comment-action

Copy link
Member

@jsnwesson jsnwesson left a comment

Choose a reason for hiding this comment

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

Looks good to me! From my understanding, aside from (1) a refactor to use double-quotes, (2) importing get_schema_view instead of get_swagger_view (which just adds a public attribute), and (3) requirement updates, there's just the Python version update in the conf.py and defining the build for readthedocs.

@deborahgu
Copy link
Member Author

Looks good to me! From my understanding, aside from (1) a refactor to use double-quotes, (2) importing get_schema_view instead of get_swagger_view (which just adds a public attribute), and (3) requirement updates, there's just the Python version update in the conf.py and defining the build for readthedocs.

yep! Thanks for that, I didn't clarify in the comment.

@deborahgu deborahgu merged commit c1f253d into main Aug 26, 2025
5 checks passed
@deborahgu deborahgu deleted the dkaplan1/APER-3932_fix-swagger-generated-api-docs-page-for-program-intents-view branch August 26, 2025 17:57
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.

3 participants