Skip to content

Ep lep workflow#873

Merged
kpsherva merged 3 commits into
CERNDocumentServer:masterfrom
kpsherva:ep-lep-workflow
Jul 22, 2026
Merged

Ep lep workflow#873
kpsherva merged 3 commits into
CERNDocumentServer:masterfrom
kpsherva:ep-lep-workflow

Conversation

@kpsherva

Copy link
Copy Markdown
Contributor

No description provided.

@kpsherva kpsherva moved this to In review 🔍 in Sprint Q3 2026 ☀️ Jul 22, 2026
@palkerecsenyi palkerecsenyi self-assigned this Jul 22, 2026
Comment thread invenio.cfg
"dd13404c-bcd6-4b15-aeef-38d678c61ff1": {
# faser
"label": "EP approval", # shown in UI buttons/headings
"referee_group": "cds-ph-ep-publication", # CERN e-group slug

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Here we need to add the committee_name. I guess for all of them it will be the same:

Suggested change
"referee_group": "cds-ph-ep-publication", # CERN e-group slug
"referee_group": "cds-ph-ep-publication", # CERN e-group slug
"committee_name": "EP Board",

@kpsherva kpsherva Jul 22, 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.

in the case of the migrator is not relevant - here we need just the prefix, but when we actually set it up on the cds-rdm then it will matter

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes indeed, on dev/sandbox/prod we will use the configmap to set this. But I'm just mentioning it here since it would be useful for us to have the correct labels on our local instances in case we're testing it.

@kpsherva
kpsherva merged commit 87dc078 into CERNDocumentServer:master Jul 22, 2026
3 checks passed
@github-project-automation github-project-automation Bot moved this from In review 🔍 to To release 🤖 in Sprint Q3 2026 ☀️ Jul 22, 2026
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