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

Switch to 404 for SUSE Manager settings #2722

Merged
merged 2 commits into from
Jun 27, 2024
Merged

Conversation

dottorblaster
Copy link
Contributor

Just switching to 404 since now 403 has a differentic semantic meaning for our frontend 👍

Verified

This commit was signed with the committer’s verified signature.
dottorblaster Alessio Biancalana
@dottorblaster dottorblaster requested a review from CDimonaco June 27, 2024 10:11
@dottorblaster dottorblaster self-assigned this Jun 27, 2024
Copy link
Member

@CDimonaco CDimonaco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
🇮🇹

Copy link
Member

@nelsonkopliku nelsonkopliku left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! Finally!

Verified

This commit was signed with the committer’s verified signature.
dottorblaster Alessio Biancalana
@dottorblaster dottorblaster force-pushed the 404-suse-manager-settings branch from 4e9f6eb to f6ac079 Compare June 27, 2024 11:08
@dottorblaster dottorblaster merged commit 62be0fe into main Jun 27, 2024
26 checks passed
@dottorblaster dottorblaster deleted the 404-suse-manager-settings branch June 27, 2024 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants