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

Remove reservee language from reservations #1499

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

matti-lamppu
Copy link
Collaborator

@matti-lamppu matti-lamppu commented Jan 22, 2025

🛠️ Changelog

  • Use user.get_preferred_language() instead

🧪 Test plan

  • Automated tests

🚧 Dependencies

🎫 Tickets

@matti-lamppu matti-lamppu added improvement Improves an existing feature breaking Requires frontend changes labels Jan 22, 2025
@matti-lamppu matti-lamppu self-assigned this Jan 22, 2025
@matti-lamppu matti-lamppu force-pushed the remove-reservee-language branch from bf3483c to 2dab451 Compare January 22, 2025 06:14
@matti-lamppu matti-lamppu marked this pull request as ready for review January 22, 2025 06:20
@matti-lamppu matti-lamppu requested review from ranta and vergama January 22, 2025 06:20
@matti-lamppu matti-lamppu force-pushed the remove-reservee-language branch from a323fdd to 12ad5bd Compare January 23, 2025 06:40
Use user.preferred_language instead
@matti-lamppu matti-lamppu force-pushed the remove-reservee-language branch from 12ad5bd to e527894 Compare January 31, 2025 07:48
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
C Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@matti-lamppu
Copy link
Collaborator Author

/ff

@github-actions github-actions bot merged commit e527894 into main Jan 31, 2025
18 of 19 checks passed
@github-actions github-actions bot deleted the remove-reservee-language branch January 31, 2025 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Requires frontend changes improvement Improves an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants