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

Feature/update decidim v0.23.5 #223

Merged

Conversation

ayuki-joto
Copy link
Collaborator

@ayuki-joto ayuki-joto commented Apr 17, 2021

🎩 What? Why?

本家 decidim の追従

📌 Related Issues

📋 Subtasks

  • Add CHANGELOG upgrade notes, if required
  • If there's a new public field, add it to GraphQL API
  • Add documentation regarding the feature
  • Add/modify seeds
  • Add tests
  • Another subtask

@takahashim
Copy link
Collaborator

対応ありがとうございます。
db/migrate/*.rb 以下の中身を見ると、DBのカラムを追加するだけではなく中身の更新も行うようなので、いったん更新した後に古いバージョンに戻そうするとうまく動かないかもしれないので要注意ですね…。

@ayuki-joto
Copy link
Collaborator Author

これが、動く環境作成して、一度検証しないとですね

@takahashim
Copy link
Collaborator

このv0.23.5用ブランチを見ているのですが、Rubyのバージョンを2.7系に指定されているのは意図的でしょうか?(Gemfile.lockやDockerfile、.github/workflows/rails-test.ymlなどを見ています)

本家のv0.23.5ではGemfile.lockと.ruby-versionを見る限り、2.6.6を使っているようだったので、揃えておいた方が安全そうに見えました。 https://github.com/decidim/decidim/tree/v0.23.5

@@ -232,5 +232,8 @@
Rails.application.config.i18n.available_locales = Decidim.available_locales
Rails.application.config.i18n.default_locale = Decidim.default_locale

## Set default comments limit. It's used in Decidim::Comments component. Default value is 100.
Rails.application.config.default_comments_limit = 5
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

↑環境変数で変えられるようにしてみます

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

お願いします!

ayuki-joto and others added 3 commits September 17, 2021 11:30
Use ENV var to define default_comments_limit
moved to `app/assets/javascripts/decidim/decidim_awesome/editors/legacy_quill_editor.js.es6`.
`legacy_quill_editor.js.es6` should be loaded after decidim files
@ayuki-joto ayuki-joto merged commit 4e17361 into codeforjapan:develop Oct 11, 2021
@ayuki-joto ayuki-joto deleted the feature/update-decidim_v0.23.5 branch February 21, 2022 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DecidimAwesomeのエディタでHTMLタグ編集機能を復活させる
3 participants