-
Notifications
You must be signed in to change notification settings - Fork 18
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
Fix/ Submission Revision Stage: update field readers accordingly #2091
Conversation
We try to not use replacement=True because that will always post an edit and run the date process function that syncs all the child invitations. Sometimes PCs make some changes that don't affect the child invitations so we don't want to be re-running the expensive date process function. |
for field in content: | ||
if field in hidden_field_names: | ||
content[field]['readers'] = [venue_id, self.venue.get_authors_id('${{4/id}/number}')] | ||
if field not in hidden_field_names and invitation_content.get(field, {}).get('readers', []): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@melisabok I reverted back to adding readers to hidden fields. However, I also now check if the invitation exists and if a field has readers that shouldn't be hidden anymore, I remove the readers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This would prevent the issue we had with NAACL today.
This PR removes
hide_fields
from the Revision invitation and allows to hide fields only using the Post_Submission.This PR also adds all possible content fields to the
hide_fields
dropdown (we were not adding thesecond_deadline_additional_options
)This issue has happened twice, once to MIDL and once to AutoML.I believe what happened was both venues had an abstract deadline, which created the/Revision
invitations. At somepoint, the pdf was hidden for all submissions. Then the Post Submission was used to unhide the pdfs. Later on, they re-run the Revision stage to allow authors to revise their papers, but they used the default name/Revision
. Since pdfs werehidden in these invitations, pdfs started being hidden after authors edited their submission.This PR checks if the field is inhide_fields
and if it isn't, it removes the readers.**Edit: I have now also noticed that if we usedreplacement=True
when posting revision invitations, we would not havethis issue. I am unsure why we sometimes usereplacement=True
and sometimes we usereplacement=False
**