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.

Sign up for GitHub

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

Jump to bottom

Make slash optional in GDPR path #1353

Merged
merged 2 commits into from
Sep 24, 2024
Merged

Make slash optional in GDPR path #1353

merged 2 commits into from
Sep 24, 2024

Conversation

matti-lamppu
Copy link
Collaborator

🛠️ Changelog

  • See title. Also validate that given uuid is correct.

🧪 Test plan

  • Automated tests

🚧 Dependencies

  • None

🎫 Tickets

  • None

@matti-lamppu matti-lamppu added the fix Fixes a bug label Sep 23, 2024
@matti-lamppu matti-lamppu self-assigned this Sep 23, 2024
@matti-lamppu matti-lamppu marked this pull request as ready for review September 23, 2024 12:06
@City-of-Helsinki City-of-Helsinki deleted a comment from github-actions bot Sep 24, 2024
@matti-lamppu matti-lamppu enabled auto-merge (rebase) September 24, 2024 06:40
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@matti-lamppu matti-lamppu merged commit 110854b into main Sep 24, 2024
6 checks passed
@matti-lamppu matti-lamppu deleted the gdpr-optional-slash branch September 24, 2024 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ranta ranta ranta approved these changes

@vergama vergama vergama approved these changes

Assignees

@matti-lamppu matti-lamppu

Labels
fix Fixes a bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@matti-lamppu @vergama @ranta