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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport 'Fix notifications page when vapid is not available' to v0.27 #10940

Conversation

ahukkanen
Copy link
Contributor

馃帺 What? Why?

Backport #10286 to v0.27

鈾ワ笍 Thank you!

@ahukkanen ahukkanen added backport Pull Requests that are a backport for a fixed bug module: core type: fix PRs that implement a fix for a bug labels May 31, 2023
@alecslupu alecslupu force-pushed the backport/0.27/fix-notifications-page-when-va-10286 branch from dcda433 to fd5f703 Compare June 14, 2023 11:23
@alecslupu alecslupu added this to the 0.27.4 milestone Jun 14, 2023
* User's group endorsement no longer disappears after personal endorsement removed

* Fixed group endorsement removal when personal endorsement removed & tests

* test-fixes

* Fix the notification settings when vapid keys are not present

---------

Co-authored-by: JoonasAapro <110532525+JoonasAapro@users.noreply.github.com>
@alecslupu alecslupu force-pushed the backport/0.27/fix-notifications-page-when-va-10286 branch from fd5f703 to dd3bb5f Compare June 14, 2023 21:40
@alecslupu alecslupu added this to Pending review from Product in Maintainers via automation Jun 14, 2023
@alecslupu alecslupu moved this from Pending review from Product to Backport pending in Maintainers Jun 14, 2023
Copy link
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 see that there are some changes that seems that have not relation to this particular PR (on .github/actions/module-rspec/action.yml and also on the Gemfile.lock files. Is that intentional? There was a failing spec related to those?

@ahukkanen
Copy link
Contributor Author

I don't know, @alecslupu has force pushed to this branch which overrides the original changes. Maybe Alex can take on this?

@alecslupu
Copy link
Contributor

I don't know, @alecslupu has force pushed to this branch which overrides the original changes. Maybe Alex can take on this?

On it. There were some issues on the rubocop side, and also on the Gemfile ...
I will let you know

@andreslucena andreslucena merged commit 9aa32e2 into release/0.27-stable Jun 15, 2023
46 checks passed
Maintainers automation moved this from Backport pending to Done Jun 15, 2023
@andreslucena andreslucena deleted the backport/0.27/fix-notifications-page-when-va-10286 branch June 15, 2023 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport Pull Requests that are a backport for a fixed bug module: core type: fix PRs that implement a fix for a bug
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants