Skip to content

Add --google-takeout-flagged-labels#51

Open
poldy79 wants to merge 1 commit into
rgladwell:masterfrom
poldy79:feature/spefify-flagged-labels
Open

Add --google-takeout-flagged-labels#51
poldy79 wants to merge 1 commit into
rgladwell:masterfrom
poldy79:feature/spefify-flagged-labels

Conversation

@poldy79

@poldy79 poldy79 commented May 10, 2022

Copy link
Copy Markdown
Contributor

By default only Important mails are marked as flagged. Add an option to
provide a comma separated list of labels to specify which mails will be
marked as flagged.

By default only Important mails are marked as flagged. Add an option to
provide a comma separated list of labels to specify which mails will be
marked as flagged.
Comment thread imap_upload.py
google_takeout_language="en",
debug=False,
google_takeout_flagged_labels="",
debug=False

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Does this even run?
Shouldn't debug=False be debug=False, (with a comma) as it's not the latest parametre?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

sorry - had to rebase and did not test... will fix that..

Comment thread imap_upload.py
self.google_takeout_first_label = google_takeout_first_label
self.google_takeout_label_priority = google_takeout_label_priority
self.google_takeout_language = google_takeout_language
self.google_takeout_flagged_labels = google_takeout_flagged_labels.split(',')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What about someone using:

--google-takeout-flagged-labels='"My spaced label","My spaced with commas, label", "NonSpacedLabel"' ?

Please check csv_file in the current code where I implemented a parser for these edge-corner cases.

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.

2 participants