-
Notifications
You must be signed in to change notification settings - Fork 31
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
feat: Create group base detail page #1329
base: master
Are you sure you want to change the base?
Conversation
b3e1ebb
to
3d3b03e
Compare
title={budgetNameAndEdit} | ||
subtitle={`${enterpriseGroup?.acceptedMembersCount} accepted members`} | ||
/> | ||
<Card.Section className="pt-1 x-small">Created on</Card.Section> |
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.
Need to add in created to the serializer, will add this value in with my next ticket
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.
looks good so far! I have some feedback regarding handling errors
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1329 +/- ##
=======================================
Coverage 85.67% 85.67%
=======================================
Files 578 583 +5
Lines 12730 12790 +60
Branches 2667 2713 +46
=======================================
+ Hits 10906 10958 +52
- Misses 1764 1771 +7
- Partials 60 61 +1 ☔ View full report in Codecov by Sentry. |
setButtonState('complete'); | ||
close(); | ||
setShowToast(true); | ||
setToastMessage('Group name updated'); |
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.
in addition to these states, I think we should also trigger a re-render for the group name card. Currently the group name stays the same unless the user triggers a refresh.
Screen.Recording.2024-10-09.at.1.39.54.PM.mov
61d9d90
to
4ec4d2a
Compare
Description
Creates the card for the groups detail page. Includes functionality for editing the group name and deleting the group.
https://2u-internal.atlassian.net/browse/ENT-9506
Test plan
For all changes
Only if submitting a visual change