Skip to content
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

Fix #139: Add Loading component in MediaBox #170

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

wangc9
Copy link

@wangc9 wangc9 commented May 10, 2024

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Bug fix
  • Why was this change needed? (You can also link to an open issue here)
    Fix 🐛 Bug Report: Add a loading in the media uploader #139, after clicking the upload button for the first time, the Loading component replaces the button until the media upload is finished.

After clicking the upload button for the first time, the Loading
component replaces the button until the media upload is finished
Copy link

vercel bot commented May 10, 2024

@wangc9 is attempting to deploy a commit to the Listinai Team on Vercel.

A member of the Team first needs to authorize it.

@nevo-david
Copy link
Contributor

Awesome! I will check it!

@nevo-david
Copy link
Contributor

@wangc9 I have checked the PR, but it doesn't work well.
You added the loader only if there is no images

When media is being loaded by SWR, the Loading component is shown instead
of directly displaying the "no media" page
@wangc9
Copy link
Author

wangc9 commented May 23, 2024

@nevo-david That's a really good point. I've now changed it so that the Loading component will also show when the box is reading media from the database. So now, the "no media" page will only show when there is nothing in the database. If there is something, however, the Loading component will show until all the media have been retrieved from the database.

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

Successfully merging this pull request may close these issues.

🐛 Bug Report: Add a loading in the media uploader
2 participants