-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
chore(admin): move constants into global constants #18609
Conversation
Size Change: -55.8 kB (-4%) Total Size: 1.48 MB
ℹ️ View Unchanged
|
I can't load the admin panel with these changes, it seems to be because of the references to
|
0456a96
to
7dc754b
Compare
7dc754b
to
8252ca0
Compare
Good catch. I've changed the menu initialization to be lazy, to work around the fact that |
What does it do?
First step to move the
useSettingsMenu
hook to TS, to make the individual bits a bit smaller. This moves the constants into the global constants file, because: 1) it always has been weird to overwrite a constant file of a hook in EE 2) the settings menu is global.