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

Prepopulate with filter values #26

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jonmifsud
Copy link
Member

This is meant to take advantage of association filters once implemented as discussed in the this thread.

Allowing new entries created through the Association UI to pre-populate the filtered fields using the provided values.

@nitriques
Copy link

@jonmifsud Could this be turned on/off ?

@jonmifsud
Copy link
Member Author

At the moment no, I was assuming if you're filtering a 'create new' button you'd want any entries created from that create new button to be pre-populated with the required values.

If I want Articles from the "News" section creating a new one should pre-fill that field with news. But if you think it makes sense to be optional we just have to see where's best to store the 'prefill / no-prefill' information. I'm thinking with the filtering options which have been added to the core PR would be the most appropriate place. but open to suggestions.

@nitriques
Copy link

you'd want any entries created from that create new button to be pre-populated with the required values.

I do no think so...

I totally understand your point and it is valid. But I sometimes deal with clients that are not super tech savvy and this would confuse them...

@jonmifsud
Copy link
Member Author

Understand your point too. Would you think this should be a site-wide or a field setting?

@nitriques
Copy link

Would you think this should be a site-wide or a field setting?

Thought question. I think I would rather see it at a section level (just like filtering).

@nilshoerrmann
Copy link
Contributor

I'm not up-to-date with the Symphony core development: is this ready to be pulled or does the code still need to be reviewed and adjusted?

@nitriques
Copy link

@jonmifsud needs to add a setting for this.

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.

None yet

3 participants