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 option to download a published authors CSV from the console #2279

Merged
merged 4 commits into from
Aug 21, 2024

Conversation

briangow
Copy link
Contributor

This adds another download option on top of what was introduced in #2270 . It gives the option to download all of the published author objects and related data.

I've included all the base PublishedAuthor fields and have linked to a couple of other relevant fields (user_id, project_id, etc.). I'm happy to add / remove fields as desired.

@tompollard
Copy link
Member

Thanks! Looks good to me, once the style issue is resolved.

Not sure we need author.pk (it might even be the same as author.id?).

@tompollard
Copy link
Member

Oh sorry, one small thing:

', '.join([a.name for a in author.affiliations.all()]), might need a bit more thought because affiliations include commas. It will be difficult to work out where one affiliation ends and the other starts.

@briangow
Copy link
Contributor Author

Thanks @tompollard , I've removed the primary key from the CSV output and now delimit multiple affiliations by using a semi-colon.

@tompollard
Copy link
Member

Addresses one of the points in: #2277

@tompollard tompollard merged commit 5556e8b into dev Aug 21, 2024
8 checks passed
@tompollard tompollard deleted the bg_add_download_published_authors branch August 21, 2024 16:38
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