Skip to content

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

Merged
andreslucena merged 4 commits intorelease/0.27-stablefrom
backport/0.27/fix-notifications-page-when-va-10286
Jun 15, 2023
Merged

Backport 'Fix notifications page when vapid is not available' to v0.27#10940
andreslucena merged 4 commits intorelease/0.27-stablefrom
backport/0.27/fix-notifications-page-when-va-10286

Conversation

@ahukkanen
Copy link
Copy Markdown
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
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 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
Copy Markdown
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
Copy Markdown
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

@alecslupu alecslupu requested a review from andreslucena June 15, 2023 06:42
@andreslucena andreslucena merged commit 9aa32e2 into release/0.27-stable 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

None yet

Development

Successfully merging this pull request may close these issues.

3 participants