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: TNL-11173 Authoring API is v0 for now #33644

Merged
merged 2 commits into from
Nov 3, 2023

Conversation

bszabo
Copy link
Contributor

@bszabo bszabo commented Nov 1, 2023

Description

This ticket contains work from the TNL-11173 JIRA task.

The changes made here move existing authoring API capabilities from their former v1-based routes to v0-routes. This is to emphasize that the current API is experimental and expected to be deprecated in short order. It's also consistent with othern TNL work exposing public APIs under v0-based 'authoring' routes on our AWS API Gateway.

Supporting information

Link to other information about the change, such as Jira issues, GitHub issues, or Discourse discussions.
Be sure to check they are publicly readable, or if not, repeat the information here.

Testing instructions

Use the same testing procedures that applied when the studio API routes were initially created. The only change that applies is that the routes are now v0 routes instead of v1 routes. Continuing to make requests off of the v1 routes should result in 404 (not found) errors.

Deadline

This PR is essential for carrying out the end-to-end tests we need to verify that our API Gateway configuration is actually working. It is therefore time sensitive.

Other information

@bszabo bszabo force-pushed the bszabo/TNL-11173-authoring-api-at-v0 branch from 51a4abb to 2d0c205 Compare November 3, 2023 14:14
@bszabo bszabo requested a review from jesperhodge November 3, 2023 14:23
@bszabo bszabo merged commit 5d6e925 into master Nov 3, 2023
51 of 62 checks passed
@bszabo bszabo deleted the bszabo/TNL-11173-authoring-api-at-v0 branch November 3, 2023 15:49
@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot
Copy link
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

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