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

Make hidden Type of Service field editable again #1354

Merged
merged 1 commit into from
Jan 28, 2025

Conversation

johanib
Copy link
Contributor

@johanib johanib commented Jan 27, 2025

Prior to this change, the Type of Service field was hidden and not
editable because users could be confused by certain internal SURF types,
such as SURF or Recommended.
This change hides those types from the UI and ensures they cannot be
modified from SP Dashboard.

Fixes #1322

@johanib johanib force-pushed the feature/1322-fix-service-type branch from 573fe05 to 7f2ff82 Compare January 27, 2025 07:11
@johanib johanib force-pushed the feature/1322-fix-service-type branch from 7f2ff82 to 3347b0a Compare January 27, 2025 07:29
@johanib johanib changed the title DRAFT 1322 - fix type of service Make hidden Type of Service field editable again Jan 27, 2025
@johanib johanib marked this pull request as ready for review January 27, 2025 07:30
@johanib johanib force-pushed the feature/1322-fix-service-type branch from 3347b0a to 3f7eecd Compare January 27, 2025 08:15
@johanib johanib requested a review from MKodde January 27, 2025 08:50
Copy link
Contributor

@MKodde MKodde left a comment

Choose a reason for hiding this comment

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

Looks great @johanib !

I only performed a code review, did not test this in the browser. If you'd like me to do that, please ping me for a quick peer review at your workstation.

Prior to this change, the Type of Service field was hidden and not
editable because users could be confused by certain internal SURF types,
such as SURF or Recommended.
This change hides those types from the UI and ensures they cannot be
modified from SP Dashboard.

Fixes #1322
@johanib johanib force-pushed the feature/1322-fix-service-type branch from 3f7eecd to ebb18d6 Compare January 27, 2025 10:33
@johanib johanib merged commit cd05e69 into main Jan 28, 2025
3 checks passed
@johanib johanib deleted the feature/1322-fix-service-type branch January 28, 2025 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Type of service field is emptied when a non existing value is already present in manage
2 participants