-
-
Notifications
You must be signed in to change notification settings - Fork 392
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
Rename system command names for consistency #12852
Rename system command names for consistency #12852
Conversation
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.
Apart from my suggestion you also have a conflict that you need to solve, can you check it out please?
decidim-system/spec/commands/decidim/system/register_organization_spec.rb
Outdated
Show resolved
Hide resolved
@greenwoodt can you check the git conflict? you need to resolve it with the command line |
…decidim into fix/rename-page-seed-command
Rebased the branch with Develop and seemed to solve the issue with the CI. Feel free to review. |
@greenwoodt we have conflicts again, sorry :( I will stop merging until we finish this one. |
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.
Heads up! There was a bad merge related to #12286
We need to leave the organization name translatable. Can you check it out please? I think with my suggestions should be enough
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
Outdated
Show resolved
Hide resolved
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 didn't catch the underscore before, sorry!
No worries! |
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
…_pages.rb Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
🎩 What? Why?
While working on footer revamp in #12592 I noticed it was quite difficult to find the Page & Topic commands which seeded the DB. To provide more clarity in the future on the location of commands which seed the DB in relation to the creation of this feature/module, we have renamed them and certain calls such as
RegisterOrganisation
accordingly.📌 Related Issues
populate_help.rb
tostatic_pages_seeds.rb
#12842Testing
Find the following file paths in your installation
decidim-system/app/commands/decidim/system/create_default_help_pages.rb
+decidim-system/app/commands/decidim/system/create_organization.rb
📷 Screenshots