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

add HasRouteNamePrefix core trait #329

Merged
merged 1 commit into from
Oct 2, 2024
Merged

add HasRouteNamePrefix core trait #329

merged 1 commit into from
Oct 2, 2024

Conversation

atmonshi
Copy link
Member

@atmonshi atmonshi commented Oct 2, 2024

No description provided.

Copy link

what-the-diff bot commented Oct 2, 2024

PR Summary

  • Introduced Route Name Management Tool to BoltPlugin Class
    Updated the BoltPlugin class with a component, referred to as 'HasRouteNamePrefix', which allows it to manage route name prefixes more efficiently.

  • Improved URL Generation for Consistent Routing
    Updated the process used to generate URLs for routes in the FormResource section. The new process leverages the route name prefix from the BoltPlugin class, resulting in more consistency in the routing procedure.

  • Modified URL Creation Process on Various Pages
    Updated the method of creating URLs on the EditForm, ListForms, and ViewForm pages. The changes ensure these pages now use the prefixed route names in their URL creation process, promoting greater standardization.

  • Enhanced Code Maintenance through Centralized Route Name Generation
    Improved the maintainability of the code by gathering the generation of route names under one umbrella - the BoltPlugin class. This centralization reduces the code complexity, decreasing potential errors and increasing efficiency.

@atmonshi atmonshi merged commit de64279 into 3.x Oct 2, 2024
6 checks passed
@atmonshi atmonshi deleted the route-name-prefix branch October 2, 2024 15:56
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