Skip to content

fix: set the collection slug to 'page-menus'#172

Merged
kevinsmaster5 merged 1 commit intomainfrom
fix-page-menu-collection-slug
Nov 13, 2025
Merged

fix: set the collection slug to 'page-menus'#172
kevinsmaster5 merged 1 commit intomainfrom
fix-page-menu-collection-slug

Conversation

@kevinsmaster5
Copy link
Contributor

Closes #152

Changes proposed in this pull request:

  • Revises the slug in the access array to use the PageMenu collection slug

Things to check

  • For any logging statements, is there any chance that they could be logging sensitive data?
  • Are log statements using a logging library with a logging level set? Setting a logging level means that log statements "below" that level will not be written to the output. For example, if the logging level is set to INFO and debugging statements are written with log.debug or similar, then they won't be written to the otput, which can prevent unintentional leaks of sensitive data.

Security considerations

There are no security implications

@kevinsmaster5 kevinsmaster5 marked this pull request as ready for review November 13, 2025 20:44
@kevinsmaster5 kevinsmaster5 requested review from a team as code owners November 13, 2025 20:44
@kevinsmaster5 kevinsmaster5 merged commit c9f052a into main Nov 13, 2025
6 checks passed
@kevinsmaster5 kevinsmaster5 deleted the fix-page-menu-collection-slug branch November 13, 2025 20:59
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.

Fix access collection config for PagesMenus

2 participants