Skip to content

Disable Admin Password Expiration And Fix Sample Data Upgrade Order#1393

Merged
markshust merged 9 commits intomarkshust:release/nextfrom
aimanecouissi:release/next
Feb 9, 2026
Merged

Disable Admin Password Expiration And Fix Sample Data Upgrade Order#1393
markshust merged 9 commits intomarkshust:release/nextfrom
aimanecouissi:release/next

Conversation

@aimanecouissi
Copy link
Contributor

This change disables admin password expiration in the development environment by setting the password lifetime to 0, so admin users are not forced to regularly change their passwords while developing.

It also moves bin/magento setup:upgrade to run immediately after sampledata:deploy. Running the upgrade only at the end cause sample data to not be properly applied and its modules appear as disabled. Running it right after deploying sample data ensures all sample data modules are correctly registered before continuing with the rest of the setup.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Dec 12, 2025
@what-the-diff
Copy link

what-the-diff bot commented Dec 12, 2025

PR Summary

  • Implement Command to Adjust Security
    Through this update, a command was added that changes the administrator's security password lifespan to zero in the initial setup script. This allows for better control and increased security measures, enhancing the overall robustness of the system.

@dosubot dosubot bot added the feature label Dec 12, 2025
Removed redundant setup:upgrade command before activating sample data.
@markshust
Copy link
Owner

I didn't see this PR so already fixed the setup:upgrade issue, however I'll pull in your other change. Thanks for this!

@markshust markshust merged commit a64fcdd into markshust:release/next Feb 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants