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 a script to create 1Password vaults from a LastPass export CSV #18

Merged
merged 5 commits into from
Jan 4, 2023

Conversation

scottisloud
Copy link
Collaborator

This PR adds a script that creates 1Password vaults based on the contents of the grouping column in a LastPass export. This script does not create 1Password items.

The value of this script would be for people who prefer, or have already used, our web-based LastPass importer. For example, they can:

  1. Import their complete LastPass data into a single 1Password vault using our web importer
  2. run this script using the same export data as the input
  3. use a 1Password desktop application to move items from the single vault to the appropriate newly-created vault based on the tag our importer applied.

This may be the best path for people with a large quantity of LastPass Secure Notes, which our web importer can ingest but our import script does not.

@scottisloud scottisloud self-assigned this Jan 4, 2023
Copy link
Member

@ag-adampike ag-adampike left a comment

Choose a reason for hiding this comment

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

Looking good so far! The only thing I can see that's missing is a reference to the new script in README.md.

@scottisloud
Copy link
Collaborator Author

Looking good so far! The only thing I can see that's missing is a reference to the new script in README.md.

Updated readme in e0a6b88

@jbsoliman
Copy link
Contributor

Looks good so far to me as well!

You could probably clean up that output to read a bit cleaner to spit out created vault names line by line; this could look as simple as

print(f"The following 1Password vaults were created:") print("\n".join(created_vault_list))

@scottisloud
Copy link
Collaborator Author

Looks good so far to me as well!

You could probably clean up that output to read a bit cleaner to spit out created vault names line by line; this could look as simple as

print(f"The following 1Password vaults were created:") print("\n".join(created_vault_list))

Right on. Fixed up that bit of feedback text and tidied up a few other parts of the readme.

@scottisloud scottisloud marked this pull request as ready for review January 4, 2023 23:02
@scottisloud scottisloud merged commit d64e4d3 into main Jan 4, 2023
@scottisloud scottisloud deleted the scoot/add-lastpass-only-create-vault branch January 9, 2023 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants