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

Patch to display user list in setting #88

Open
quanvn opened this issue Jun 24, 2015 · 1 comment
Open

Patch to display user list in setting #88

quanvn opened this issue Jun 24, 2015 · 1 comment
Assignees
Milestone

Comments

@quanvn
Copy link

quanvn commented Jun 24, 2015

Hi @nutso, I make this small patch in the setting views to allow admin to select user from user list instead of finding and entering user id. Please check and consider merge it into your main release. Thanks
File affect app/views/settings/_recurring_tasks_settings.html.rb

  1. Add these at the begining of file
    <%-
    users = User.active.all.sort_by {|e| e[:lastname]}
    -%>
  2. replace the

with
<%= select_tag("settings[journal_attributed_to_user]",
options_from_collection_for_select(users, :id, :name, settings['journal_attributed_to_user']),
:include_blank => true) %>

@nutso nutso added this to the v2.0.0 milestone Jun 25, 2015
@nutso nutso self-assigned this Jun 25, 2015
nutso added a commit that referenced this issue Jun 30, 2015
@nutso
Copy link
Owner

nutso commented Jun 30, 2015

develop branch confirmed in 2.x, still need to test on 3.x

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