[ENG-5028] [ENG-5920] Preprints Affiliation Project PR (BE)

[cslzchen]

Purpose

This is the project BE PR for Preprints Institution Affiliation.

It comprises the following PRs that were merged into the feature branch

• ENG-5829 Add Institutional Affiliation mix-in to Preprints
  • https://github.com/CenterForOpenScience/osf.io/pull/10649
• ENG-5844 Add preprint institution affiliation endpoint
  • https://github.com/CenterForOpenScience/osf.io/pull/10655
• ENG-5845 Preprint Institutions Relationship
  • https://github.com/CenterForOpenScience/osf.io/pull/10659
• ENG-5846 and ENG-5847 Add affiliated_institutions to preprint list and detail endpoint
  • https://github.com/CenterForOpenScience/osf.io/pull/10684
• [Bug-fix] Fix Self Link for Preprint Affiliated Institutions
  • https://github.com/CenterForOpenScience/osf.io/pull/10698
• ENG-5908 Add Test Confirming Preprint Affiliated Institution Data is Sent to SHARE
  • https://github.com/CenterForOpenScience/osf.io/pull/10700
• ENG-5966 2.0.1 BE: Update permission to support edit as a WRITE contributor
  • https://github.com/CenterForOpenScience/osf.io/pull/10741

Changes

• Model
  • Add AffiliatedInstitutionMixin mixin to Preprint model and overrides affiliated_institutions with related name.
  • Add affiliation add/remove to PreprintLog
  • Remove deprecated save=False in add_affiliated_institution()
• API
  • New Endpoint:
  • PreprintInstitutionsList: this endpoint is read-only, use existing InstitutionSerializer and new PreprintInstitutionPermissionList
  • PreprintInstitutionsRelationship: this endpoint only allows GET and PUT, uses new PreprintsInstitutionsRelationshipSerializer and updated WriteOrPublicForRelationshipInstitutions
  • Serializer:
  • New PreprintsInstitutionsRelationshipSerializer for PreprintInstitutionsRelationship
  • Move InstitutionRelated from nodes to institutions
  • Add affiliated_institutions to PreprintSerializer
  • Move assertion checks into one helper method handle_author_assertions()
  • Permission
  • Move WriteOrPublicForRelationshipInstitutions from nodes to base and add a special case for Preprint
  • Add PreprintInstitutionPermissionList: this is a permission for read-only endpoint
  • Verified that WRITE user can't do the following three:
    • Manage contributor (except for removing themselves), this doesn't require any code change, since contributor has its own endpoint and permission
    • Can't change author assertion, this is in handle_author_assertions()
    • Can't withdraw preprint, this doesn't require any code change either, since it's also handled by existing code outside of Preprints.can_edit().
  • Moved helper methods from nodes to institutions and added a new helper update_institutions_if_user_associated()
• Share
  • Add tests to make sure that direct affiliation info is sent to share
• Unit tests
• API Doc: see https://github.com/CenterForOpenScience/developer.osf.io/pull/63

QA Notes

See project QA Planning

Documentation

See https://github.com/CenterForOpenScience/developer.osf.io/pull/63, make sure we release documentation to developer.osf.io too.

Side Effects

N/A

Ticket

Project: https://openscience.atlassian.net/browse/ENG-5028 BE: https://openscience.atlassian.net/browse/ENG-5920

[cslzchen]

Note: there is some weird but unrelated test failures for my local env