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

feat: update decidim v0.27.9 #634

Merged

Conversation

ayuki-joto
Copy link
Collaborator

🎩 What? Why?

decidim v0.27.9へのupdate です

📌 Related Issues

  • Related to #?
  • Fixes #?

📋 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

📷 Screenshots (optional)

Description

Copy link
Collaborator

@takahashim takahashim left a comment

Choose a reason for hiding this comment

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

public/sw.js.*のファイルがいくつか更新されてるようですが、これって何でしたっけ…?
(public/sw.jsとpublic/sw.js.mapはそもそも登録されてない?)
それ以外は特に問題なさそうです。

@ayuki-joto
Copy link
Collaborator Author

public/sw.js.*のファイルがいくつか更新されてるようですが、これって何でしたっけ…? (public/sw.jsとpublic/sw.js.mapはそもそも登録されてない?) それ以外は特に問題なさそうです。

これ、迷ったんですが、precompile 時に勝手に生成されるもので、既存のものがcommitされてたんで、一応更新しておくかくらいです!

@takahashim
Copy link
Collaborator

どちらかというと.gz.brがついてない方も登録するか(こうすると差分が人間にもわかるかも?)、全部登録しないかのどちらかが良さそうな気がしますが、どうなんでしょうか。

@ayuki-joto
Copy link
Collaborator Author

compileごとに再生成されてそうなので、含めないが無難そうですね

@ayuki-joto ayuki-joto merged commit e90272a into codeforjapan:main Oct 7, 2024
2 checks passed
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.

2 participants