Skip to content

Backport 'Correct smtp_settings keys type' on release/0.22-stable#6905

Merged
tramuntanal merged 2 commits intodecidim:release/0.22-stablefrom
Quentinchampenois:revert/smtp-settings-0.22-stable
Nov 25, 2020
Merged

Backport 'Correct smtp_settings keys type' on release/0.22-stable#6905
tramuntanal merged 2 commits intodecidim:release/0.22-stablefrom
Quentinchampenois:revert/smtp-settings-0.22-stable

Conversation

@Quentinchampenois
Copy link
Copy Markdown
Contributor

@Quentinchampenois Quentinchampenois commented Nov 20, 2020

This reverts commit e86f959.

🎩 What? Why?

I apologize for this reverted backport proposition, we just noticed this morning a smtp settings issue introduced by #6664

UPDATE
This PR is no longer a revert but a backport of fixes added in #6908

📌 Related Issues

sauloperez added a commit to CoopCat-Confederacio-de-Cooperatives/decidim-coopcat that referenced this pull request Nov 20, 2020
There's a fix in decidim/decidim#6908 for the
regression introduced in decidim/decidim#6664,
but its revert done in decidim/decidim#6905 is
not merged either. We can't wait any longer so I'm doing the revert
myself.
* Revert for backport

This reverts commit ef0f3b3.

* Backport 'Correct smtp_settings keys type'
@Quentinchampenois Quentinchampenois changed the title Revert "Backport 'Backport smtp settings correction' to v0.22 (#6878)" WIP Backport 'Correct smtp_settings keys type' on release/0.22-stable Nov 24, 2020
@Quentinchampenois Quentinchampenois changed the title WIP Backport 'Correct smtp_settings keys type' on release/0.22-stable Backport 'Correct smtp_settings keys type' on release/0.22-stable Nov 24, 2020
@tramuntanal tramuntanal merged commit a46bf2a into decidim:release/0.22-stable Nov 25, 2020
@tramuntanal
Copy link
Copy Markdown
Contributor

Merging as the erb-linter error is not related with this PR and it doesn't change any erb file

sauloperez added a commit to CoopCat-Confederacio-de-Cooperatives/decidim-coopcat that referenced this pull request Nov 30, 2020
Now we can point back to upstream because
decidim/decidim#6938,
decidim/decidim#6905 and
decidim/decidim#6893 have been merged. There
were also other bug fixes in between.
davidbeig pushed a commit to CoopCat-Confederacio-de-Cooperatives/decidim-coopcat that referenced this pull request Dec 2, 2025
There's a fix in decidim/decidim#6908 for the
regression introduced in decidim/decidim#6664,
but its revert done in decidim/decidim#6905 is
not merged either. We can't wait any longer so I'm doing the revert
myself.
davidbeig pushed a commit to CoopCat-Confederacio-de-Cooperatives/decidim-coopcat that referenced this pull request Dec 2, 2025
Now we can point back to upstream because
decidim/decidim#6938,
decidim/decidim#6905 and
decidim/decidim#6893 have been merged. There
were also other bug fixes in between.
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