Skip to content

feat(ep-approval): add read-only flag - #975

Open
zzacharo wants to merge 1 commit into
CERNDocumentServer:masterfrom
zzacharo:ep-approval-read-only
Open

zzacharo wants to merge 1 commit into
CERNDocumentServer:masterfrom
zzacharo:ep-approval-read-only

Conversation

@zzacharo

@zzacharo zzacharo commented Sep 16, 2026 •

Copy link
Copy Markdown
Contributor

read-only enrolled communities are displaying past approved records but disallow new submissions

closes CERNDocumentServer/cds-migrator-kit#600

Existing EP approved record

Screenshot 2026-09-16 at 15 34 39

Non-requested EP approved record

Screenshot 2026-09-16 at 15 34 50

Comment thread invenio.cfg
},
"1c46ceab-af68-4502-a436-cdf5a4cfbbe0": {
# isolde
"read_only": True, # only allow read-only access to this community

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can add this as well to LEP experiments

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need to add this in our openshift environments too, lets document it in CERNDocumentServer/cds-migrator-kit#600

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it would be good if you add it locally as well, for completeness and to avoid confusion

@zzacharo zzacharo Sep 23, 2026 •

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@kpsherva Can you check the updated list?

Comment on lines +230 to +231
return {
"can_submit": False,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you explain why this dict need to be returned in this form? Out of curiosity, I want to understand how it works

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

basically this is a UI serialization on what is needed to render the component.

* read-only enrolled communities are displaying past approved records but disallow new submissions
@zzacharo
zzacharo force-pushed the ep-approval-read-only branch from 5df94a2 to 2faf262 Compare September 23, 2026 09:54

This branch has not been deployed

No deployments
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.

migration: evaluate communities with records with EP reportnumbers that they should not have the EP approval workflow enabled

3 participants