Skip to content

Conversation

azbarcea
Copy link

Fixes #1056

  • Created the convention for the singlefile to always use the file stem (the filename without extension) as the name of the collection.
  • Added .vscode and .env to .gitignore

Comment on lines -64 to -72
path = os.path.abspath(expand_path(path))
try:
path_glob = path % "*"
except TypeError:
# If not exactly one '%s' is present, we cannot discover
# collections because we wouldn't know which name to assign.
raise NotImplementedError

placeholder_pos = path.index("%s")
Copy link
Member

Choose a reason for hiding this comment

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

See my comment here: #1056 (comment)

Copy link
Author

@azbarcea azbarcea Apr 21, 2023

Choose a reason for hiding this comment

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

I think removing this block is solves your initial intent, if I understand it right (will come back with more explanation in #1056 )

I can also confirm that the import from singlefile was now successful with current .patch.

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.

Question/Feature request: Sync/merge contacts from a Google/iCloud .vcf export/backup
2 participants