krishnaacharyaa / wanderlust

WanderLust is a MERN travel blog website 🚀 This project is aimed to help people to contribute in open source, upskill in react and master git.
https://wanderlust-beta.vercel.app/
MIT License
221 stars 1.24k forks source link

Added Loading Skeleton for the AddBlog Section #446

Closed AMS003010 closed 4 months ago

AMS003010 commented 4 months ago

Summary

This PR provides a Loading Skeleton for the AddBlog section so that the user has a seamless experience

Description

AddBlogPostSkeleton Component has been added and has been used to replace the previous Loader component that was used.

Images

screenshot

Issue(s) Addressed

Closes #445
Enter the issue number of the bug(s) that this PR fixes

Prerequisites

vercel[bot] commented 4 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
wanderlust ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 3:38pm
wanderlust-backend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jul 10, 2024 3:38pm
github-actions[bot] commented 4 months ago

Umm... did someone forget to read the style guide? Fix that PR title and let's try again! @AMS003010 Don't worry, it happens to the best of us! Check out our contributor guidelines for more details.

krishnaacharyaa commented 4 months ago

image

So now according to your logic, whereever the RequireAuth is used I'll get your this skeleton loading ....!!

AMS003010 commented 4 months ago

Sorry, I'll fix that

AMS003010 commented 4 months ago

Actually umashri_meti wants to fix the rest.... Can we do that in a single PR

krishnaacharyaa commented 4 months ago

@umashri_meti

krishnaacharyaa commented 4 months ago

@MetiUmashri kindly work on top of this PR if you wanna do a combined PR @AMS003010 yes it is possible, please aim to close this soon. As there is nothing much to be done

krishnaacharyaa commented 4 months ago

image

So now according to your logic, whereever the RequireAuth is used I'll get your this skeleton loading ....!!

@AMS003010 @MetiUmashri this is still not solved even in the newly raised PR :)