-
Notifications
You must be signed in to change notification settings - Fork 275
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
feature/remove_currently_at #5121
Conversation
We will need to upgrade documentation (Upgrade guide and others) to remove |
done |
@nqb Just merge it if all is fine. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New GUI need to display option to enable/disable configurator, not the case right now in Advanced section.
@cgx, can you take a look at this ?
@jrouzierinverse: could you add upgrade instructions and/or write a script that delete old currently-at file for users doing upgrades ? I updated our post-upgrade template added at end of upgrade guide to remove reference to currently-at file in Database schema section. |
@nqb I've added the toggle button to enable/disable the Configurator. |
Thanks @cgx, I will wait upgrade instructions or script migration from @jrouzierinverse now. |
Done |
I add another part to your upgrade notes in Upgrade guide. Let me know if it's fine for you. |
Description
Remove the need for the currently_at file
Impacts
The configurator
NEWS file entries
Enhancements
UPGRADE file entries
The file /usr/local/pf/conf/currently-at is no longer needed it can be removed.
rm -rf /usr/local/pf/conf/currently-at