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

Application Skeleton Implementation #195

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Ishimwe7
Copy link

@Ishimwe7 Ishimwe7 commented Oct 11, 2024

Description
This PR introduces skeleton loading components throughout the application to improve user experience by providing visual feedback during data fetching. These skeleton loaders mimic the structure of the final content, making the interface more intuitive and responsive while the content is loading.

Description of Task Completed

  • Implemented skeleton loading components across key areas of the application.
  • Skeleton loaders are designed to represent the final structure of the content, enhancing the visual feedback to users during loading times.
  • Integrated loaders with existing features to ensure a smooth transition when content is being fetched from the backend.

Related Issue
#161

NavbarMembers

traineeApplicant

editApplicantInfo

trash

Copy link

vercel bot commented Oct 11, 2024

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

Name Status Preview Comments Updated (UTC)
atlp-devpulse-fn ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 14, 2024 9:25pm

@mugishaj092 mugishaj092 self-requested a review October 11, 2024 10:03
@Ishimwe7 Ishimwe7 added Ready For Merge and removed Ready For Review Extra attention is needed labels Oct 14, 2024
Comment on lines 26 to 29
// useEffect(() => {
// props.getAllCycles();
// }, []);

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Get rid of all unnecessary comments

@@ -20,6 +21,7 @@ const TrainneeDetails = (props: any) => {
}, [ID]);

const traineeDetails = oneTraineeDetails.data;
console.log("Trainee: ", traineeDetails);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove all logs

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let me remove them

Copy link

codeclimate bot commented Oct 14, 2024

Code Climate has analyzed commit d34ee76 and detected 0 issues on this pull request.

View more on Code Climate.

@ncutixavier
Copy link
Contributor

This PR has conflicts

@Ishimwe7
Copy link
Author

This PR has conflicts

Let me resolve them

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

Successfully merging this pull request may close these issues.

5 participants