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

Default author assignment setting doesn't work with large user sets #25

Closed
timkelty opened this issue Aug 24, 2017 · 6 comments
Closed

Comments

@timkelty
Copy link

I have a guest entries section where thousands of existing users have author privileges, so they all show up in the dropdown…but only the first 100 of them, so I can never select the default user I want.

Thinking this selection should probably be an elementselect modal instead.

@brandonkelly
Copy link
Member

Hey Tim, can you send a DB backup to support@craftcms.com, and point out which user you can't access? Not seeing anything in the code that would limit the dropdown to 100 users.

@timkelty
Copy link
Author

timkelty commented Sep 5, 2017

@brandonkelly Ah - I this was for the 1.x version, where default limit is 100, not null.

That said, PHP is still likely going to crap out with large datasets like this, so I think it should still be converted to an elementselect modal. I'll work on a PR if you agree.

@brandonkelly
Copy link
Member

Sorry didn’t realize this was for v1. Can fix it there. Will be harder to fix for v2 since the settings have been redesigned as an Editable Table, which currently doesn't support element select columns (cc craftcms/cms#811).

@brandonkelly brandonkelly reopened this Sep 5, 2017
brandonkelly added a commit that referenced this issue Sep 5, 2017
@timkelty
Copy link
Author

timkelty commented Sep 5, 2017

Thanks. Will brainstorm on how to address this for v2 once this project is on Craft 3.

@brandonkelly
Copy link
Member

Decided to just keep it a normal dropdown for v1, since in v1 the dropdown actually doubles as the way to enable/disable allowing guest entry submission for the section. So swapping it with a element select field would add confusion, unless we redesigned the settings to be more like they are in v2.

@timkelty
Copy link
Author

Will brainstorm on how to address this for v2 once this project is on Craft 3.

@brandonkelly can you reopen this? It is still an issue for Craft 3. The only way I can adjust settings is via project-config, as I have too many users for the settings templates to render.

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

No branches or pull requests

2 participants