-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[SLO] Add/edit form mark optional fields #175807
Conversation
/ci |
🤖 GitHub commentsExpand to view the GitHub comments
Just comment with:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just one small change and then I think we are ready to LGTM.
export function OptionalText() { | ||
return ( | ||
<EuiText size="xs" color="subdued"> | ||
{i18n.translate('xpack.observability.slo.sloEdit.optionalLabel', { | ||
defaultMessage: 'Optional', | ||
})} | ||
</EuiText> | ||
); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we move this into a seperate file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
Pinging @elastic/obs-ux-management-team (Team:obs-ux-management) |
💚 Build Succeeded
Metrics [docs]Module Count
Async chunks
History
To update your PR or re-run it, just comment with: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
* main: use build hash in FTR tests [Security Solution] Fix moderate typo (elastic#175883) [Fleet] Fix conflicting dynamic template mappings for intermediate objects (elastic#175970) [Visualize] Prevent overwriting managed content (elastic#175274) [SLO] Add/edit form mark optional fields (elastic#175807) skip failing test suite (elastic#175984) [data views] Provide method of excluding data tiers when getting field list (elastic#167946) [Dataset quality] State management (elastic#174906) [Cloud Security] add is_internal config option for outputs (elastic#175546)
## Summary Fixes elastic/observability-dev#3049 Add/edit form mark optional fields !! <img width="593" alt="image" src="https://github.com/elastic/kibana/assets/3505601/909155f4-4b8a-4444-a2ad-ac0770c56929">
## Summary Fixes elastic/observability-dev#3049 Add/edit form mark optional fields !! <img width="593" alt="image" src="https://github.com/elastic/kibana/assets/3505601/909155f4-4b8a-4444-a2ad-ac0770c56929">
Summary
Fixes https://github.com/elastic/observability-dev/issues/3049
Add/edit form mark optional fields !!