Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

MUMMNG-2401 Adds WRS Statement notification #432

Merged
merged 1 commit into from
Mar 21, 2016

Conversation

vertein
Copy link
Contributor

@vertein vertein commented Mar 21, 2016

Took the liberty and made it dissmissable. I put a comment in the JIRA, won't merge until I hear one way or the other.

@timlevett
Copy link
Contributor

👍

@apetro
Copy link
Contributor

apetro commented Mar 21, 2016

👍. Yes, since it won't automatically dismiss when users consume their WRS statement of benefits, if they even have one, it should be end-user dismissable.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 28.178% when pulling e26601c on MUMMNG-2401-WRS-Statement into 2e5cbe3 on master.

"Facstaff - Former Appointment"],
"title" : "2015 WRS Statements of Benefits are posted. Check if you have one.",
"actionURL" : "/portal/p/university-staff-benefits-statement",
"actionAlt" : "Access the list of available WRS Statement",
Copy link
Contributor

Choose a reason for hiding this comment

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

Trivial: Some kind of agreement-in-number problem with this text. Maybe just doesn't matter since the text isn't used for anything?

@vertein vertein force-pushed the MUMMNG-2401-WRS-Statement branch from e26601c to 7581740 Compare March 21, 2016 16:27
@apetro
Copy link
Contributor

apetro commented Mar 21, 2016

Mockup:

mockup of notification

@vertein
Copy link
Contributor Author

vertein commented Mar 21, 2016

Fixed plural issue

vertein added a commit that referenced this pull request Mar 21, 2016
MUMMNG-2401 Adds WRS Statement notification
@vertein vertein merged commit 11b43c2 into master Mar 21, 2016
@vertein vertein deleted the MUMMNG-2401-WRS-Statement branch March 21, 2016 16:28
@jhanstra
Copy link
Contributor

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants