lightdash icon indicating copy to clipboard operation
lightdash copied to clipboard

fix: dashboard mentions not working sometimes and added loading state

Open anshul45 opened this issue 1 year ago • 1 comments

Closes: https://github.com/lightdash/lightdash/issues/10666

Description:

In this pull request https://github.com/lightdash/lightdash/pull/11212, I didn't add a loading state. I created a loading component that waits for useOrganizationUsers and useSpace to resolve before displaying the CommentForm.

Reviewer actions

  • [ ] I have manually tested the changes in the preview environment
  • [ ] I have reviewed the code
  • [ ] I understand that "request changes" will block this PR from merging

anshul45 avatar Aug 22 '24 17:08 anshul45

Deploy request for peaceful-bassi-cbf284 pending review.

Visit the deploys page to approve it

Name Link
Latest commit 6335455fe29bcc1eb045a6bafb7899a87b3c0fd7

netlify[bot] avatar Aug 22 '24 17:08 netlify[bot]

@anshul45 changed hook to return undefined instead of an empty array when the data was not ready. Also made the loading /skeleton just on the input so it fits nicely then there are other comments.

mentions

ZeRego avatar Sep 02 '24 13:09 ZeRego

working on it

anshul45 avatar Sep 02 '24 14:09 anshul45

working on it

@anshul45 I already pushed the changes 👍

ZeRego avatar Sep 02 '24 14:09 ZeRego

:tada: This PR is included in version 0.1237.4 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

lightdash-bot avatar Sep 02 '24 14:09 lightdash-bot

Thanks for merging my request

anshul45 avatar Sep 02 '24 15:09 anshul45