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

fix(dedicated.server): dont call orderable endpoint if not orderable #12152

Merged
merged 2 commits into from
Jun 28, 2024

Conversation

qpavy
Copy link
Contributor

@qpavy qpavy commented Jun 26, 2024

ref: INC0049502

Question Answer
Branch? master
Bug fix? yes
New feature? no
Breaking change? no
Tickets Fix #INC0049502
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • [] Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • [ ] Breaking change is mentioned in relevant commits

Description

Related

ref: INC0049502

Signed-off-by: Quentin Pavy <quentin.pavy.ext@ovhcloud.com>
@qpavy qpavy requested a review from a team as a code owner June 26, 2024 14:19
@github-actions github-actions bot added universe-bare-metal-cloud bug Something isn't working labels Jun 26, 2024
darsene
darsene previously approved these changes Jun 26, 2024
 ref: INC0049502

Signed-off-by: soufien mhelhali <mhalheli.soufien@gmail.com>
Copy link

sonarcloud bot commented Jun 27, 2024

darsene
darsene previously approved these changes Jun 27, 2024
@ghyenne ghyenne changed the base branch from master to release/incidents-w26 June 28, 2024 14:23
@ghyenne ghyenne dismissed stale reviews from darsene and mhelhali-soufien June 28, 2024 14:23

The base branch was changed.

@ghyenne ghyenne merged commit 6de11e2 into release/incidents-w26 Jun 28, 2024
15 checks passed
@ghyenne ghyenne deleted the fix/INC0049502-3 branch June 28, 2024 14:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants