Skip to content

[SD-1641] [SD-1662] Fix 'Publication infomation' typo in group_publication_metadata label and Previewer role shouldn't be able to bypass site restriction#758

Open
vincent-gao wants to merge 2 commits intodevelopfrom
SD-1641-change-label
Open

[SD-1641] [SD-1662] Fix 'Publication infomation' typo in group_publication_metadata label and Previewer role shouldn't be able to bypass site restriction#758
vincent-gao wants to merge 2 commits intodevelopfrom
SD-1641-change-label

Conversation

@vincent-gao
Copy link
Copy Markdown
Contributor

@vincent-gao vincent-gao commented Mar 25, 2026

Jira

https://digital-vic.atlassian.net/browse/SD-1641
https://digital-vic.atlassian.net/browse/SD-1662

Changes

  1. Fixes a typo.
  2. Previewer role shouldn't be able to bypass site restriction

Fix 'Publication infomation' typo in group_publication_metadata label
@vincent-gao vincent-gao changed the title [SD-1641] Fix 'Publication infomation' typo [SD-1641] Fix 'Publication infomation' typo in group_publication_metadata label Mar 25, 2026
@vincent-gao vincent-gao self-assigned this Mar 25, 2026
Revoke 'bypass site restriction' permission from previewer role.
$tideCoreOperation = new TideCoreOperation();
$tideCoreOperation->assignCkeditorTemplatesPermission();
}

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should this be in here? We're fixing a couple of things?

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.

yes. @anthony-malkoun , sorry, I forgot to update the title and description.
I have update the PR to include https://digital-vic.atlassian.net/browse/SD-1662

@vincent-gao vincent-gao changed the title [SD-1641] Fix 'Publication infomation' typo in group_publication_metadata label [SD-1641] [SD-1662] Fix 'Publication infomation' typo in group_publication_metadata label and Previewer role shouldn't be able to bypass site restriction Mar 30, 2026
Copy link
Copy Markdown

@anthony-malkoun anthony-malkoun left a comment

Choose a reason for hiding this comment

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

Thanks for updating!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants