Chore/refactor duplications - #92
Open
pablo-schmeiser wants to merge 16 commits into
Open
Conversation
- Add non-field error handling and improve already present error handling in shift creation - Remove green hghlighting for valid, but optional form fields and removing green checkmark from optional form select fields
This is a combination of 5 commits. - chore: Create Mixins for shared, inherited behavior - chore: Refactor shared membership inheritance - chore: Register models to admin view in helper function - chore: Refactor URL patterns to use shared organization CRUD helper - chore: Refactor URL patterns to utilize organization_crud_paths for events and shifts
This is a combination of 3 commits. - fix: Group details were not reachable due to same url - fix: Correct inheritance order for OrganizationCreateUpdateMixin - fix: Refactor delete view classes to fix inheritance issue
This is a combination of 3 commits. - chore: Refactor form templates to use shared remove/restore button component - chore: Refactor often reused buttons to deduplicate code and fix modal backdrop behavior NOTE: Modal is now top-level to correct behavior. Temporary patch for iOS was removed and should be tested on iOS in the future. - chore: Refactor modal trigger components to reduce duplication and improve maintainability
This is a combination of 9 commits. - chore: Add coverage configuration and update dependencies for testing - test: Add set_language to tests - test: Add unit tests for user and membership forms to improve coverage - test: Add unit tests for user authentication and email confirmation flows - fix: Update SSO login condition to check if OAUTH is enabled - test: Add unit tests for SSO login and protected content views - test: Consolidate DOM manipulation tests and improve structure - Introduced a new `dom-harness.cjs` file to centralize DOM-related utilities for testing. - Created various test files for different components, including formset, modal handlers, navigation, sliders, save and restore functionality, and theme toggle. - Refactored class list and element creation logic to reduce duplication across tests. - Enhanced test setups to better simulate user interactions and DOM events. - Improved assertions for clarity and accuracy in testing expected behaviors. - test: Add unit tests for BaseMixin and permission handling in views - docs: Add Testing Quality Scorecard documentation
- Removed member_shift_summary template and integrated its functionality into shift_users_summary. - Updated templates to use shift_users_summary for displaying user shifts. - Added non-org participants to summary. - Introduced tests for shift_users_summary functionality and filtering behavior.
pablo-schmeiser
commented
Aug 5, 2026
pablo-schmeiser
left a comment
pablo-schmeiser
left a comment
Collaborator
Author
There was a problem hiding this comment.
Since this PR looks imposing, the tests and changes to the CSS are basically irrelevant, and the rest is mostly straight forward refactorings and updated .po files for localization.
Collaborator
Author
There was a problem hiding this comment.
While an improvement, this could still be solved nicer.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This pull request introduces a new user theme preference feature, improves test coverage and reporting, and adds comprehensive documentation and metrics for testing quality. It also includes UI refinements for login pages and codebase cleanup.
User Theme Preference Feature:
theme_preferencefield to theUsermodel with options for 'light', 'dark', and 'auto', including a migration and admin registration update. [1] [2] [3]Testing Quality and Coverage Improvements:
.coveragercfor coverage configuration and enabled thedjango-coverage-pluginin development and test dependencies. [1] [2]docs/testing_quality.mdwith a detailed scorecard for behavior-focused testing quality metrics, and linked it from development and README docs. [1] [2] [3] [4]accounts/tests/test_auth_views.py.UI and Template Refinements:
login-surfaceclass to relevant login form containers for consistent theming. [1] [2] [3] [4] [5]Codebase Cleanup:
Usermodel to use the sharedregister_modelsutility.Documentation Improvements: