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: clear controlled affiliations #913

Merged
merged 1 commit into from
Nov 24, 2022
Merged

Conversation

jma
Copy link
Contributor

@jma jma commented Nov 23, 2022

Co-Authored-by: Johnny Mariéthoz Johnny.Mariethoz@rero.ch
Co-Authored-by: Pascal Repond pascal.repond@rero.ch

Why are you opening this PR?

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

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?

* Closes rero#807.
* Removes controlled affiliations because sometimes a controlled
  affiliation becomes uncontrolled.

Co-Authored-by: Johnny Mariéthoz <Johnny.Mariethoz@rero.ch>
Co-Authored-by:  Pascal Repond <pascal.repond@rero.ch>
@jma jma force-pushed the maj-fix-affiliations branch from 2f33a44 to e175d7d Compare November 23, 2022 08:50
@PascalRepond PascalRepond self-requested a review November 23, 2022 09:50
@jma jma requested review from rerowep and vgranata November 23, 2022 10:57
@jma jma marked this pull request as ready for review November 23, 2022 10:57
@jma jma merged commit 4ed6af9 into rero:staging Nov 24, 2022
# 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.

Controlled affiliation should be cleared when affiliation is modified to empty or unrecognized value
4 participants