Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Since Pods 2.9.4: Restricted Access within Templates defaults to Yes #6916

Closed
danmaby opened this issue Sep 22, 2022 · 2 comments
Closed

Since Pods 2.9.4: Restricted Access within Templates defaults to Yes #6916

danmaby opened this issue Sep 22, 2022 · 2 comments

Comments

@danmaby
Copy link
Contributor

danmaby commented Sep 22, 2022

Description

When creating a new template or editing an existing one in 2.9.4, the Restricted Access settings all default to Yes. When unchecking these settings and updating, they default to Yes.

Version

2.9.4

Testing Instructions

  1. Publish a new Template with no Restricted Access
  2. Review Template
  3. See restrictions applied
  4. Remove restrictions
  5. Update
  6. See restrictions applied again

Screenshots / Screencast

pods-2 9 4

Possible Workaround

Temporarily downgrading to 2.9.3 restores expected behaviour.

Site Health Information

No response

Pods Package

No response

@oldrup
Copy link

oldrup commented Sep 22, 2022

Mmm, as this obviously only really affects people without access rights, it goes unseen by me, the admin, until some non-admin user complains:
image

I've spend some time creating a screencast, somewhat redundant to the fine testing instructions by @danmaby but here it is:
https://www.loom.com/share/09931098c9e048f5b33f097a4079d06c

Workaround: disabling the restrictions actually seems to work, even though they still look enabled in the backend.

Is this related to "Fixed: Pod/Group/Field configurations being saved now get their checkbox boolean options properly enforced as 0/1 and sent so they are not reverted to the default value on next load. #6485 (@sc0ttkclark)" ?

Bjarne

@sc0ttkclark
Copy link
Member

@oldrup it was unrelated to that it appears

Fixed this in the end by just making this into a legitimate pod object (since it was an internal pod but fields were not defined the same way).

960d4e4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

4 participants