-
Notifications
You must be signed in to change notification settings - Fork 117
Conversation
api.codeConfig.allowLockedModelChanges = false; // after this point in the Actionhero boot lifecycle, locked models cannot be changed | ||
await loadSampleProfiles(); |
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.
The story called for this to not be awaited and run in the "background". However, I'm concerned about creating parallel queries when using SQLite (which ui-config is guaranteed to be using).
At the very least, we now are using a "regular" Initializer, not CLSInitializer so failures are OK and the transaction won't be polluted by an import failure here.
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.
That's a good point but I'm also worried it might take a while to launch.
At a minimum, let's log before we do that so it's clear what is happening on the console.
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.
added in f89ddaa
This has conflicts. |
Adds a Button in
ui-config
to import all sample Records.Also automatically tries to pre-load all the existing sample records at boot when using
ui-cofig
Checklists
Development
Impact
Please explain any security, performance, migration, or other impacts if relevant:
Code review