Skip to content

allow new authors schema that supports institutions#2741

Draft
melisabok wants to merge 9 commits intomasterfrom
feature/new-authors-schema
Draft

allow new authors schema that supports institutions#2741
melisabok wants to merge 9 commits intomasterfrom
feature/new-authors-schema

Conversation

@melisabok
Copy link
Copy Markdown
Member

  • Add new setting in the venue request form: authors_with_instutions that enables the new schema, the venue must force the submissions to have author profiles only.
  • Remove deprecated settings in the submission stage: create_groups, create_review_invitations
  • Support authorids in the class Note when using the new schema.
  • Add a tests

@xkopenreview
Copy link
Copy Markdown
Contributor

all invitations which alter reader of authorids will need to be changed

@xkopenreview
Copy link
Copy Markdown
Contributor

assuming there are 2 invitations for a submitted paper:
invitation1 for reorder only (built based on institution selected during submission)
invitation2 for reorder + institution change

after user update the note using invitation2, invitation1 must be updated to match updated result in the note
otherwise it will fail the validation of reorder only invitation1

@melisabok
Copy link
Copy Markdown
Member Author

all invitations which alter reader of authorids will need to be changed
This should be done.

assuming there are 2 invitations for a submitted paper:
invitation1 for reorder only (built based on institution selected during submission)
invitation2 for reorder + institution change

Should I support the invitation2?

after user update the note using invitation2, invitation1 must be updated to match updated result in the note
otherwise it will fail the validation of reorder only invitation1

I don't think both invitations should be active at the same time, do you see an scenario when this is not true?

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