Skip to content

test adding fields to Submission Change Before Reviewing#2865

Draft
celestemartinez wants to merge 9 commits intomasterfrom
chore/test-delete-true
Draft

test adding fields to Submission Change Before Reviewing#2865
celestemartinez wants to merge 9 commits intomasterfrom
chore/test-delete-true

Conversation

@celestemartinez
Copy link
Copy Markdown
Member

This test is designed to test adding fields to the Submission_Change_Before_Reviewing invitation. The invitation is initially created with the following content in the invitation.edit.note.content:

{
    'authors': {
        'readers': [
            'ABCD.cc/2025/Conference',
            'ABCD.cc/2025/Conference/Submission${{4/id}/number}/Authors'
        ]
    },
    'authorids': {
        'readers': [
            'ABCD.cc/2025/Conference',
            'ABCD.cc/2025/Conference/Submission${{4/id}/number}/Authors'
        ]
    },
    'pdf': {
        'readers': {
            'param': {
                'const': {
                    'delete': True
                }
            }
        }
    }
}

In the test, I try to add readers to the subject_areas field using the Submission_Change_Before_Reviewing/Restrict_Field_Visibility. However, after the edit, the Submission_Change_Before_Reviewing ends up with the following content instead, and there is no way to delete the pdf readers.

{
    'authors': {
        'readers': [
            'ABCD.cc/2025/Conference',
            'ABCD.cc/2025/Conference/Submission${{4/id}/number}/Authors'
        ]
    },
    'authorids': {
        'readers': [
            'ABCD.cc/2025/Conference',
            'ABCD.cc/2025/Conference/Submission${{4/id}/number}/Authors'
        ]
    },
    'subject_areas': {
        'readers': [
            'ABCD.cc/2025/Conference',
            'ABCD.cc/2025/Conference/Submission${{4/id}/number}/Authors'
        ]
    }
}

@celestemartinez
Copy link
Copy Markdown
Member Author

@carlosmondra you can use this test to see the issue with { "delete" : True }. Let me know if there is anything you have questions about.

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