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.

Sign up for GitHub

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

Jump to bottom

Mixed inlines/formsets breaks on lack of permissions #1434

Closed
blms opened this issue Aug 9, 2023 · 1 comment
Closed

Mixed inlines/formsets breaks on lack of permissions #1434

blms opened this issue Aug 9, 2023 · 1 comment
Assignees
@blms
Labels
🐛 bug Something isn't working

Comments

@blms
Copy link
Contributor

blms commented Aug 9, 2023

If a user does not have permissions to view all of the inlines on a page using mixed inlines/formsets, it results in an IndexError as get_fieldsets_and_inlines gets a smaller number of inlines than expected.
@blms blms added the 🐛 bug Something isn't working label Aug 9, 2023
@blms blms self-assigned this Aug 9, 2023
blms added a commit that referenced this issue Aug 9, 2023
@blms
Respect perms in mixed inlines/formsets (#1434)
f0a99f0
blms added a commit that referenced this issue Aug 9, 2023
@blms
Merge pull request #1435 from Princeton-CDH/bugfix/1434-inlines-forms…
9dfe138 
…ets-permissions

Respect perms in mixed inlines/formsets (#1434)
@blms
Copy link
Contributor Author

blms commented Aug 9, 2023

Confirmed fixed in QA

@blms blms closed this as completed Aug 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@blms blms

Labels
🐛 bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@blms