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
Standardise current_user
call within Commands tasks (Participatory Processes)
#12840
Standardise current_user
call within Commands tasks (Participatory Processes)
#12840
Conversation
…m:greenwoodt/decidim into fix/standardize-current_user-calls/part-2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think I found a another place where to change this:
decidim-participatory_processes/app/commands/decidim/participatory_processes/admin/import_participatory_process.rb
Can you change it there 🙏🏽 ?
Can the current_user be standardised here? |
Yes, we're calling |
...controllers/decidim/participatory_processes/admin/participatory_process_copies_controller.rb
Outdated
Show resolved
Hide resolved
…ipatory_processes/admin/participatory_process_copies_controller.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
current_user
call within Commands tasks (Participatory Processes)
…ipatory_processes/admin/participatory_process_copies_controller.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
current_user
call within Commands tasks (Participatory Processes)current_user
call within Commands tasks (Participatory Processes)
🎩 What? Why?
current_user
call is standardized/refactored within the specific tasks of various Classes in the Participatory Processes module.📌 Related Issues
Testing
run
decidim-participatory_processes/spec/commands/copy_participatory_process_spec.rb
📷 Screenshots