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

Put view classes in separate files #1563

Merged
merged 9 commits into from
Jul 25, 2024
Merged

Conversation

dchiller
Copy link
Contributor

@dchiller dchiller commented Jul 16, 2024

Closes #1533.

This PR distributes view previously in views/views.py into:

  • api.py: Contains any "api" views (views with responses that are not rendered and don't fit into redirect or autocomplete view categories)
  • auth.py: Contains the "change password" view
  • autocomplete.py: Contains views used in the implementation of autocomplete fields
  • contact.py: Contains a view for the contact flatpage
  • redirect.py: Contains redirect views
  • site_stats.py: Contains views for the "item count" and "content overview" pages

For the future, it probably makes sense to manage the contact page with the flatpages app. I've added this as a low priority issue #1572.

@dchiller dchiller force-pushed the split-views-files branch from a23ac41 to 6ad61db Compare July 23, 2024 14:25
@dchiller dchiller force-pushed the split-views-files branch from 915eee9 to ad35080 Compare July 23, 2024 18:18
@dchiller dchiller marked this pull request as ready for review July 23, 2024 18:22
@dchiller dchiller requested a review from lucasmarchd01 July 23, 2024 18:29
@dchiller dchiller requested a review from lucasmarchd01 July 24, 2024 16:53
@dchiller dchiller force-pushed the split-views-files branch from aa870ed to c121369 Compare July 24, 2024 16:59
@dchiller dchiller requested a review from lucasmarchd01 July 24, 2024 17:02
@dchiller dchiller merged commit c447fb1 into DDMAL:develop Jul 25, 2024
1 check passed
@dchiller dchiller deleted the split-views-files branch July 25, 2024 17:07
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.

Separate views into their own files
3 participants