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

Fix Incorrect URLs for Edit and Delete Actions in Unit Groups Table #2010

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

taimoorimran
Copy link
Contributor

Title:

Fix URL Routing for Edit and Delete Actions in Unit Groups Table

Description:

Summary:
This pull request resolves an issue where the edit and delete actions in the Unit Groups table were incorrectly routed to perform actions on Units instead of Unit Groups.

Details:

  • Corrected the URLs in the Unit Groups table to ensure that edit and delete actions are performed on Unit Groups rather than Units.
  • Updated route handling to accurately target the Unit Groups for these actions.

Impact:

  • Ensures that modifications and deletions are applied to the correct entity (Unit Groups) rather than mistakenly affecting Units.

Testing:

  • Verified that the edit and delete actions now correctly affect Unit Groups as intended.
  • Confirmed that no actions are mistakenly applied to Units.

This fix improves the accuracy of action routing in the Unit Groups table, ensuring proper management of unit group entries.

Copy link

what-the-diff bot commented Aug 2, 2024

PR Summary

  • Updated URLs for Edit and Delete actions in UnitGroupCrud.php
    The locations where the system processes 'Edit' and 'Delete' actions for unit groups were modified. Specifically, the URLs for these actions were changed. This doesn't alter the user experience directly, but rather helps in maintaining and managing the system effectively.

  • Added confirmation message for Delete action in UnitGroupCrud.php
    To increase the user experience and reduce accidental deletions, a confirmation message has been added to the 'Delete' action for unit groups. This means users will now be prompted to confirm their intention before they can delete a unit group.

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