Skip to content

Internal: Fix condition for the slack notification#31321

Merged
Svitlana-Dykun merged 11 commits intomainfrom
internal-fix-slack-notification
May 28, 2025
Merged

Internal: Fix condition for the slack notification#31321
Svitlana-Dykun merged 11 commits intomainfrom
internal-fix-slack-notification

Conversation

@Svitlana-Dykun
Copy link
Copy Markdown
Contributor

PR Checklist

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

Summary

This PR can be summarized in the following changelog entry:

Description

An explanation of what is done in this PR

Test instructions

This PR can be tested by following these steps:

Quality assurance

  • I have tested this code to the best of my abilities
  • I have added unittests to verify the code works as intended
  • Docs have been added / updated (for bug fixes / features)

Fixes #

@Svitlana-Dykun Svitlana-Dykun added the run-tests Runs Playwright tests on Pull request label May 27, 2025
@Svitlana-Dykun Svitlana-Dykun merged commit 12aeb80 into main May 28, 2025
53 of 54 checks passed
@Svitlana-Dykun Svitlana-Dykun deleted the internal-fix-slack-notification branch May 28, 2025 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-tests Runs Playwright tests on Pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants