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

feat: Evaluate x-viur-bonelist on default viewSkel() (light-version) #1415

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

phorward
Copy link
Member

This is a lighter version of #1384, integrating the feature only for Singleton and List.

This is a lighter version of viur-framework#1384, integrating the feature only for `Singleton` and `List`.
@phorward phorward added feature New feature or request Priority: High After critical issues are fixed, these should be dealt with before any further issues. main labels Feb 17, 2025
@phorward phorward added this to the ViUR-core v3.7 milestone Feb 17, 2025
ArneGudermann
ArneGudermann previously approved these changes Feb 18, 2025
@phorward phorward added viur-meeting Issues to discuss in the next ViUR meeting and removed viur-meeting Issues to discuss in the next ViUR meeting labels Feb 28, 2025
@phorward phorward requested a review from sveneberth March 7, 2025 16:07
bones = set(bones) if bones else set()

if allow_client_defined:
# if bonelist := current.request.get().kwargs.get(X_VIUR_BONELIST.lower()): # DEBUG
Copy link
Member Author

Choose a reason for hiding this comment

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

@sveneberth I wonder if this could also be useful for debugging, what do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request main Priority: High After critical issues are fixed, these should be dealt with before any further issues.
Projects
Status: Todo
Development

Successfully merging this pull request may close these issues.

3 participants