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

persons: link persons to source instead of MEF #996

Merged
merged 1 commit into from
May 19, 2020
Merged

Conversation

rerowep
Copy link
Contributor

@rerowep rerowep commented May 19, 2020

  • Links documents to the source authority file, such as GND or IdRef,
    through the MEF server, instead of the MEF clusterized record. This
    is needed to provide a stable link to the authority.

My PR depends on the following rero-ils's PR(s):

Co-Authored-by: Peter Weber [email protected]

Why are you opening this PR?

  • Which task/US does it implement?
  • Which issue does it fix?

Dependencies

My PR depends on the following rero-ils-ui's PR(s):

  • rero/rero-ils-ui#

How to test?

  • What command should I have to run to test your PR?
  • What should I test through the UI?

Code review check list

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

@rerowep rerowep self-assigned this May 19, 2020
@rerowep rerowep added this to the v0.9.0 milestone May 19, 2020
* Links documents to the source authority file, such as GND or IdRef,
  through the MEF server, instead of the MEF clusterized record. This
  is needed to provide a stable link to the authority.

My PR depends on the following `rero-ils`'s PR(s):

* rero/rero-ils-ui#269

Co-Authored-by: Peter Weber <[email protected]>
@rerowep rerowep force-pushed the US1241-link-gnd-idref branch from 7495228 to aa81fc9 Compare May 19, 2020 09:04
@rerowep rerowep merged commit 2f4ebd8 into dev May 19, 2020
@rerowep rerowep deleted the US1241-link-gnd-idref branch May 20, 2020 08:34
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.

1 participant