Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

documents: adapt import from HEP BEJUNE #649

Merged
merged 1 commit into from
Nov 2, 2021

Conversation

mmo
Copy link
Collaborator

@mmo mmo commented Sep 20, 2021

Why are you opening this PR?

How to test?

  • What command should I have to run to test your PR? invenio imports hepbejune /[FOLDER]/export_memoires_curated_20210913_sample.csv [FOLDER]
  • What should I test through the UI? The records are imported with the correct fields

Code review check list

  • Commit message template compliance.
  • Commit message without typos.
  • File names.
  • Functions names.
  • Functions docstrings.
  • Unnecessary commited files?
  • Extracted translations?

@mmo mmo force-pushed the mmo-adjust-bejune-import branch from 4843fcb to c93aa2e Compare October 5, 2021 13:49
Co-Authored-by: Miguel Moreira <miguel.moreira@rero.ch>
@mmo mmo force-pushed the mmo-adjust-bejune-import branch from c93aa2e to d6bb17a Compare October 5, 2021 14:04
@jma jma changed the title Mmo adjust bejune import documents: adapt import from HEP BEJUNE Nov 2, 2021
@jma jma merged commit 2463ff8 into rero:staging Nov 2, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants