-
Notifications
You must be signed in to change notification settings - Fork 1
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
Upgrade Decidim to v0.27 #490
base: master
Are you sure you want to change the base?
Conversation
3f553ab
to
122d641
Compare
db/migrate/20240905140527_change_types_in_participatory_processes.rb
Outdated
Show resolved
Hide resolved
db/migrate/20240905140527_change_types_in_participatory_processes.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.
LGTM
cce7ca2
to
169f454
Compare
f075653
to
a1f3fcf
Compare
aacf09d
to
2f93fcb
Compare
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.
Is there a test? can we document exactly what are we changing from the original class?
Can we use a decorator instead of overriding the whole class?
def url | ||
Decidim::ResourceLocatorPresenter.new(process).url&.split("?")&.first | ||
end |
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.
can you document why we need this? Process URLs don't have query parameters no?
def short_description_ca | ||
ActionController::Base.helpers.strip_tags(process.short_description["ca"]) |
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.
Why remove and take all translations instead?
🎩 What? Why?
Upgrade Decidim to v0.27
Release notes: https://github.com/CodiTramuntana/decidim/blob/release/0.27-stable/CHANGELOG.md
In this version, participatory processes has type. We have a custom module for that (decidim-type) but is not necessary anymore, so we remove the custom module.
FIXES
Example in other modal
Fix
TO-DO
Related PRs:
📷 Screenshots (optional)