Skip to content

Fix Shakapacker upgrade does not work for existing instances#10814

Merged
andreslucena merged 6 commits intodecidim:developfrom
i-need-another-coffee:fix/10800
Jul 3, 2023
Merged

Fix Shakapacker upgrade does not work for existing instances#10814
andreslucena merged 6 commits intodecidim:developfrom
i-need-another-coffee:fix/10800

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

@alecslupu alecslupu commented May 3, 2023

🎩 What? Why?

This PR attempts to fix issue reported in #10800.
It will remove bin/webpack, bin/webpack-dev-server and install the shakapacker binaries: bin/webpacker and bin/webpacker-dev-server.
While I was working on this, i have noticed that .ruby-version was not changed from 3.0.2 to 3.1.1.

📌 Related Issues

Link your PR to an issue

Testing

Generate a Decidim 0.27 application, and follow the upgrade steps.

♥️ Thank you!

@alecslupu alecslupu requested a review from a team June 4, 2023 08:57
@alecslupu alecslupu added type: fix PRs that implement a fix for a bug release: v0.28 Issues or PRs that need to be tackled for v0.28 module: core labels Jun 19, 2023
@alecslupu alecslupu marked this pull request as ready for review June 19, 2023 21:25
@andreslucena andreslucena self-assigned this Jun 20, 2023
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

I have one doubt and one suggestion, can you check it out 🙏🏽?

@alecslupu alecslupu requested a review from andreslucena June 30, 2023 00:03
Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
@alecslupu alecslupu requested a review from andreslucena June 30, 2023 15:42
@alecslupu alecslupu added type: internal PRs that aren't necessary to add to the CHANGELOG for implementers and removed type: fix PRs that implement a fix for a bug release: v0.28 Issues or PRs that need to be tackled for v0.28 labels Jul 2, 2023
Copy link
Copy Markdown
Member

@andreslucena andreslucena left a comment

Choose a reason for hiding this comment

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

👍🏽 works perfect if you actually follow all the steps of the RELEASE_NOTES 🤣

@andreslucena andreslucena added the no-backport Pull Requests that should not be backported label Jul 3, 2023
@andreslucena andreslucena merged commit 9eab73f into decidim:develop Jul 3, 2023
@alecslupu alecslupu deleted the fix/10800 branch July 3, 2023 06:05
entantoencuanto added a commit that referenced this pull request Jul 7, 2023
* feature/redesign:
  Redesign: replace cells with redesigned versions if present and update references (#11123)
  Remove duplicated constant
  Simplify logic
  Enable specs
  Fix leaflet
  fix failing specs
  Fix issues with overriding maps and loading Leaflet (#11105)
  Update decidim-proposals/app/views/decidim/proposals/proposals/new.html.erb
  Update decidim-comments/lib/decidim/comments/comments_helper.rb
  Update decidim-assemblies/spec/system/filter_assemblies_spec.rb
  Update decidim-assemblies/spec/system/filter_assemblies_spec.rb
  Document how to work locally with Elections/Votings (#10870)
  Fix Admin dashboard disappear if you are in Trustee Zone (#11111)
  Fix Shakapacker upgrade does not work for existing instances (#10814)
  Fix for sending welcome emails for new participants (#10991)
  Fix seeded trustees (#10964)
andreslucena added a commit that referenced this pull request Jul 18, 2023
* Fix Shakapacker upgrade does not work for existing instances

* Remove uneeded task

* Update the release notes

* Remove webpacker related files

* running linters

* Update RELEASE_NOTES.md

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>

---------

Co-authored-by: Andrés Pereira de Lucena <andreslucena@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

module: core no-backport Pull Requests that should not be backported type: internal PRs that aren't necessary to add to the CHANGELOG for implementers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Shakapacker upgrade does not work for existing instances

2 participants