-
Notifications
You must be signed in to change notification settings - Fork 243
Open
Labels
help wantedOpen to participation from the communityOpen to participation from the community🐍 tech: pythonInvolves PythonInvolves Python💻 aspect: codeConcerns the software code in the repositoryConcerns the software code in the repository🔧 tech: djangoInvolves DjangoInvolves Django🛠 goal: fixBug fixBug fix🟨 priority: mediumNot blocking but should be addressed soonNot blocking but should be addressed soon🧱 stack: apiRelated to the Django APIRelated to the Django API
Description
Description
WordPress/openverse-api#806 added the validateopenapischema management command primarily to confirm that trailing slashes would no longer accidentally be left off of routes. We unintentionally removed the management command in #1876, probably because it used the old schema generator and we missed that it was doing more than just checking that the schema was valid.
Fix
Add a new management command called checktrailingslashes and add it to CI in the Django checks section.
Additional context
Pinging @dhruvkb to make sure that I'm not missing something that still covers the trailing slash issue.
dhruvkb
Metadata
Metadata
Assignees
Labels
help wantedOpen to participation from the communityOpen to participation from the community🐍 tech: pythonInvolves PythonInvolves Python💻 aspect: codeConcerns the software code in the repositoryConcerns the software code in the repository🔧 tech: djangoInvolves DjangoInvolves Django🛠 goal: fixBug fixBug fix🟨 priority: mediumNot blocking but should be addressed soonNot blocking but should be addressed soon🧱 stack: apiRelated to the Django APIRelated to the Django API
Type
Projects
Status
📅 To Do