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

Editor grids permissions refactor, enhancements, and fixes #16653

Draft
wants to merge 37 commits into
base: 3.x
Choose a base branch
from

Conversation

smg6511
Copy link
Collaborator

@smg6511 smg6511 commented Nov 29, 2024

Related PR

See also #15919. This is a re-packaging of that PR; this new PR seeks to cover all editor grids across the application and separates out some work (for a future PR) that had been begun around internationalizing core names and descriptions within some of the grids.

This PR is temporarily in draft mode while I finish up some details and ensure all grids have been covered.

Formatting, style, optimization only
Changes to js, css, and Lexicon common to multiple areas of this PR
Changes to both grid list and Context editing page (general info)
Implements new permissions handling and fixes a couple other issues in the Dashboard editing panel:
- Added validation to prevent dup Dashboard names
- Styled toggles to match rest of current UI
Formatting, code style changes only
Hides actions icon for first (currently installed) package
Hides actions icon for unchanged Lexicons
Formatting, style updates only
Updates display of and ability to select row actions (gear icon, bulk actions button); includes update to getViewConfig method in base grid class
Formatting, code style changes only
Formatting, code style changes only
Final functional and minor display changes
Formatting, code style changes only
Updates display of and ability to select row actions (gear icon, bulk actions button). Also fixes index controller so users with view permissions can see the grid of Users. Lastly removes unused method for Users grid class.
Formatting, code styling changes only
Formatting, code style changes only
The new getExtrasNamespaces method was needed in places other than the main Namespaces page (via GetList); made static as well. Also updates GetList to use translatable Creator names.
Consolidate shared methods and config elements into new GridBase class
Second round/finalization of changes to this area
Remove commented out line
Updates display of and ability to select row actions (gear icon)
Formatting, code style changes only
Updates display of and ability to select row actions (gear icon). Also adjustments made to base grid class.
Remove dev logging
Formatting, code style changes only
Updates display of and ability to select row actions (gear icon). Also adjustments made to base grid class.
Formatting, code style & optimization only
Formatting, code style & optimization changes only
Formatting, code style & optimization changes only
Updates display of and ability to select row actions (gear icon), as well as display of various action buttons. Also adjustments made to base grid class.
@smg6511 smg6511 changed the title Grids permissions rework Editor grids permissions refactor, enhancements, and fixes Nov 29, 2024
@smg6511 smg6511 marked this pull request as draft November 29, 2024 04:34
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.

1 participant