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

Add check-empty-type support to explicit load calls. #584

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eigood
Copy link
Contributor

@eigood eigood commented May 8, 2023

If "load" is requested from the moqui cli, then all given data load types are always run. However, if moqui is started normally, it has special support to load certain data items only the first time, when the database is empty(checked via looking for Enumeration records).

This feature lets one specify check-empty-types, and load types. Then, always call "load" in the deployment process, and the first time through(empty database) it'll load seed,seed-initial, etc, and subsequent calls will just load seed.

If "load" is requested from the moqui cli, then all given data load
types are *always* run.  However, if moqui is started normally, it has
special support to load certain data items only the *first* time, when
the database is empty(checked via looking for Enumeration records).

This feature lets one specify check-empty-types, and load types.  Then,
always call "load" in the deployment process, and the first time
through(empty database) it'll load seed,seed-initial, etc, and subsequent
calls will just load seed.
@acetousk
Copy link
Member

I'm not sure what the objective or use case of this change is. Do you mind explaining that?

It seems like you are adding another way to specify types of data to load conditionally if the database is not loaded (through an enumeration) unless they are specified in the load parameter (where they will always load) in a command line setting.

@jonesde
Copy link
Member

jonesde commented Jul 24, 2023

There is a recently added feature to load data on start, separate from the empty db load, which sounds similar to what you describe.

Here is the commit:

9bd047b

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants