-
Notifications
You must be signed in to change notification settings - Fork 38
Add submit banner to admin page for organizations #407
Conversation
padding: 8px 8px; | ||
min-width: 200px; | ||
font-size: 16px; | ||
} |
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.
could use some space underneath of it
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.
fca3938
to
d8d33a3
Compare
const goodTitle = 'Success!' | ||
const badTitle = 'Something went wrong!' | ||
|
||
const formatBanner = (props) => { |
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 wonder if we should also optionally pass in the titles so they can be more specific to what succeeded or failed. Not a blocker though
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.
That's a good idea. It's generally more flexible that way, too. I'll add it to the scrum board though, since I'd like to ship this ASAP.
Add submit banner to admin page for organizations
Add submit banner to admin page for organizations
Pops up a basic banner on an Organization edit page's submit.
Bueno
No Bueno
@zendesk/volunteer
Fixes #402