-
Notifications
You must be signed in to change notification settings - Fork 900
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
refactor(titus/serverGroup): Reactify Titus Resize Server Group Modal #7175
refactor(titus/serverGroup): Reactify Titus Resize Server Group Modal #7175
Conversation
</div> | ||
</div> | ||
|
||
{!!errorMessage && ( |
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.
Uses formik, but only shows one error message and always shows that error message in the same spot.
} | ||
|
||
function SimpleMode({ formik, serverGroup, toggleMode }: IAdvancedModeProps) { | ||
useEffect(() => { |
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.
I love how simple and clean useEffect
can be.
); | ||
|
||
@Overridable('titus.serverGroup.CapacityDetailsSection') | ||
export class TitusCapacityDetailsSection extends React.Component<ICapacityDetailsSectionProps> { |
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.
this is still a class component because @Overridable
needs a class component
The following commits need their title changed: Please format your commit title into the form:
This allows us to easily generate changelogs & determine semantic version numbers when cutting releases. You can read more about commit conventions here. |
6630f9b refactor(core/serverGroup): Extract capacity details components to reuse across providers (spinnaker#7182) 20df06e refactor(titus/serverGroup): Reactify Titus Resize Server Group Modal (spinnaker#7175)
And also reactify the server group capacity details section.
Also give this the Changes treatment we did in Amazon provider
use formik and hooks