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 some additional documentation about record ownership during import #29

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

Conversation

jat255
Copy link

@jat255 jat255 commented Sep 13, 2024

No description provided.

@@ -75,6 +75,13 @@ Figure out the Team ID by looking at the Team from the Sysconfig panel, where th
# import in team 42, force everything to be of type "Resources" with category "6"
docker exec -it elabftw bin/console import:eln --type items --category 6 your.eln 42

By default (if no ``--userid`` setting is provided), the ownership of the items will be
determined by comparing the email addresses of users between the export and import
servers. This allows for simple migration of data from one ELabFTW server to another,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make sure to use correct case: eLabFTW

By default (if no ``--userid`` setting is provided), the ownership of the items will be
determined by comparing the email addresses of users between the export and import
servers. This allows for simple migration of data from one ELabFTW server to another,
even if users have differing ``userid`` values on the two servers. If the user that owns
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer "instance" instead of "servers". Or maybe both "instances/servers".

servers. This allows for simple migration of data from one ELabFTW server to another,
even if users have differing ``userid`` values on the two servers. If the user that owns
an item in the exported data is not present on the destination (import) server, the
importer will create users as necessary.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"importer" could be replaced by "import process", to make it cleare we're talking about the code, not the human doing the import.

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.

2 participants