-
Notifications
You must be signed in to change notification settings - Fork 43
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
✨ Add empty state for empty groups / stakeholders list #1817
Conversation
67f2236
to
04e1487
Compare
118f0cb
to
226ff5a
Compare
Signed-off-by: Ian Bolton <ibolton@redhat.com>
226ff5a
to
909e4d2
Compare
…biage Signed-off-by: Ian Bolton <ibolton@redhat.com>
909e4d2
to
7849653
Compare
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.
The Empty state bits LGTM
As @rszwajko pointed out, a bunch of our translations texts are more like string templates than translatable keys. Since this is fairly prevalent across the codebase, I don't consider this a blocking problem. Something to think about what/if we want to do something in future.
Resolves konveyor#1816 <img width="1650" alt="Screenshot 2024-04-04 at 3 25 17 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/b4c3d099-83ee-49e6-b498-0877ddd8caee"> <img width="1647" alt="Screenshot 2024-04-04 at 3 25 06 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/f2d4519d-dcb9-49c8-aab9-e0734d3f322a"> <img width="1654" alt="Screenshot 2024-04-04 at 3 24 59 PM" src="https://github.com/konveyor/tackle2-ui/assets/11218376/5861a412-b1ff-48f6-82d6-410a4a7d1411"> --------- Signed-off-by: Ian Bolton <ibolton@redhat.com>
Resolves #1816