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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport 'Verify modules are installed in StatsParticipantsCount query' to v0.26 #11158

Conversation

alecslupu
Copy link
Contributor

🎩 What? Why?

Backport #11138 to v0.26

♥️ Thank you!

@alecslupu alecslupu 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 Jul 12, 2023
@alecslupu alecslupu added this to Pending review from Product in Maintainers via automation Jul 12, 2023
@alecslupu alecslupu added this to the 0.26.8 milestone Jul 12, 2023
@andreslucena
Copy link
Member

Oh I approved it but then I saw the failing checks @alecslupu - we're using an API that doesn't exists in v0.26.

We should change it to another strategy (like with the defined? keyword)

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.

See comment

@alecslupu
Copy link
Contributor Author

Oh I approved it but then I saw the failing checks @alecslupu - we're using an API that doesn't exists in v0.26.

We should change it to another strategy (like with the defined? keyword)

I have a better idea: Implement the functionality in 0.26, as it seems that is a fix that has not been backported. Please check: #11168

@andreslucena
Copy link
Member

I have a better idea: Implement the functionality in 0.26, as it seems that is a fix that has not been backported. Please check: #11168

Ok! You got lucky because #9577 was a fix that we didn't backport to v0.26 🤣

@andreslucena
Copy link
Member

Please check: #11168

@alecslupu it was merge already, can you merge with the release branch 🙏🏽 ? Thanks

… backport/0.26/verify-modules-are-installed-i-11138
@andreslucena andreslucena merged commit 4e1e649 into release/0.26-stable Jul 12, 2023
25 checks passed
Maintainers automation moved this from Pending review from Product to Done Jul 12, 2023
@andreslucena andreslucena deleted the backport/0.26/verify-modules-are-installed-i-11138 branch July 12, 2023 17:10
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

2 participants