Add edit log managment command #414
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ekansa
approved these changes
May 3, 2024
| @@ -308,6 +320,8 @@ models, branches, thesauri, and collections intact. | |||
|
|
|||
| This mimics the 'Create Mapping File' command from the Arches Designer UI. | |||
There was a problem hiding this comment.
Let's add a link to the part of the documentation describing mapping files:
:ref:`Mapping File`
| @@ -308,6 +320,8 @@ models, branches, thesauri, and collections intact. | |||
|
|
|||
| This mimics the 'Create Mapping File' command from the Arches Designer UI. | |||
|
|
|||
| import mapping file | |||
There was a problem hiding this comment.
I think it would be good to give a little more description about when and why one may want to import a mapping file? Just a sentence or two about what role mapping files play in ETL workflows and when these command line utilities will be of use.
|
|
||
| python manage.py resources clear_edit_log | ||
|
|
||
| Removes all entries from the Arches Edit Log. |
There was a problem hiding this comment.
It would be good to know when / why it may be a good idea to purge the edit log.
| @@ -299,6 +301,16 @@ Other Data Management Commands | |||
| Removes all resources from your database, but leaves the all resources | |||
| models, branches, thesauri, and collections intact. | |||
There was a problem hiding this comment.
This seems drastic! Does this command leave a trace in the edit log? We may want to give a warning that this nukes a big chuck of your database...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
brief description of changes
Accounts for a new management command added in archesproject/arches#10837
issues addressed
further comments
This box must be checked
This box should be checked
This box should only be checked you intend to follow through on it (we can do it on our end too)
cherry-pickall commits in this PR into other branches that should have them after this PR is merged