Skip to content

Refactor javascript in decidim core (Part 2)#15064

Merged
greenwoodt merged 12 commits intodecidim:developfrom
i-need-another-coffee:chore/refactor/js-bulk
Aug 20, 2025
Merged

Refactor javascript in decidim core (Part 2)#15064
greenwoodt merged 12 commits intodecidim:developfrom
i-need-another-coffee:chore/refactor/js-bulk

Conversation

@alecslupu
Copy link
Copy Markdown
Contributor

🎩 What? Why?

This is a continuation of the work done in #15045

  • we are removing unused files
  • we are moving some of the files that we are not planning on refactoring
  • adding additional stimulus controllers

📌 Related Issues

Link your PR to an issue

Testing

Each one of the commits have been tested and reviewed individually by @greenwoodt , so the only constraint is "green pipeline"

📷 Screenshots

Please add screenshots of the changes you are proposing
Description

♥️ Thank you!

github-actions[bot]
github-actions bot previously approved these changes Aug 19, 2025
github-actions[bot]
github-actions bot previously approved these changes Aug 19, 2025
github-actions[bot]
github-actions bot previously approved these changes Aug 19, 2025
@greenwoodt greenwoodt added javascript Pull requests that update Javascript code configuration labels Aug 20, 2025
@alecslupu alecslupu marked this pull request as ready for review August 20, 2025 06:52
@greenwoodt greenwoodt self-requested a review August 20, 2025 06:57
Copy link
Copy Markdown
Member

@greenwoodt greenwoodt left a comment

Choose a reason for hiding this comment

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

Merging with a green pipeline, nice job @alecslupu!

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants