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

Added functionality to update a specific route in the web.php file if it exists. This change was made to ensure that the specified route is updated correctly during the installation process. #12

Merged
merged 5 commits into from
Jun 12, 2024

Conversation

thejmitchener
Copy link
Member

@thejmitchener thejmitchener commented Jun 12, 2024

Introduces a new functionality to update a specific route in the web.php file if it already exists. The purpose of this change is to ensure that the specified route is updated correctly during the installation process.

The NavigationInstallCommand.php now includes logic to locate the specified route in the web.php file and update it with the required modifications. This enhancement ensures that the route is correctly updated as part of the installation process, providing a more seamless experience for users.

By incorporating this new functionality, the project's installation process becomes more robust and user-friendly, as it guarantees the accurate updating of the specified route in the web.php file.

thejmitchener and others added 5 commits June 12, 2024 15:26
… it exists, otherwise display an error message. This change was made to ensure that the specified route is updated correctly during the installation process.
…for file operations and simplified the logic for updating a route in the web.php file. The changes were made to improve code readability and maintainability.
@thejmitchener thejmitchener merged commit aaaf97f into main Jun 12, 2024
10 checks passed
@thejmitchener thejmitchener deleted the update-install-command branch June 12, 2024 21:16
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