Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

[$40] Show Alerts #57

Closed
maxceem opened this issue Jul 1, 2021 · 5 comments
Closed

[$40] Show Alerts #57

maxceem opened this issue Jul 1, 2021 · 5 comments

Comments

@maxceem
Copy link
Contributor

maxceem commented Jul 1, 2021

Add Alerts column as per design https://marvelapp.com/prototype/g6g2jh7/screen/78212451

image

  • Don't add sorting and filtering to this column
  • Add the next warings for now:
    • if RB.billingAccount is not set, show BA - Not Assigned
    • if the current WP is last one, then show Last Booking Week
  • Make such warning to have max width, and if text is longer, then add ....
  • When we hover such messages with mouse we should show a Tooltip with the full message. Reuse Tooltip from here [$175] Provide Job Title and ProjectId info for teamname #38
@maxceem maxceem added CF Issues for Community Fixes Blocked P4 and removed Blocked labels Jul 1, 2021
@maxceem maxceem changed the title Show Alerts [$40] Show Alerts Jul 3, 2021
@MadOPcode
Copy link
Contributor

@maxceem how to display multiple alerts in the same cell?

@maxceem
Copy link
Contributor Author

maxceem commented Jul 3, 2021

@MadOPcode hmm, good question.

I think we can show them comma separated inside the yellow label. And inside the tooltip show them on the new lines like you did for checkboxes.

@sandhiyakavi
Copy link
Collaborator

Verified on Dev Env.

alerts_column

@maxceem
Copy link
Contributor Author

maxceem commented Jul 7, 2021

@maxceem maxceem added the PAID label Jul 7, 2021
@sandhiyakavi
Copy link
Collaborator

Verified on Prod Env with @nkumar-topcoder

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

No branches or pull requests

3 participants