Skip to content

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

@eigood
Copy link
Contributor Author

eigood commented Apr 25, 2025

I created this feature, then manually added the new lines from the microgram UOM PR, and never bothered to merge this branch back into our own fork.

I shall get back to you on whether 9bd047b works(I suspect it will, given my reading of that change).

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.

3 participants