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

Refactor/styling #35

Merged
merged 7 commits into from
Oct 23, 2023
Merged

Refactor/styling #35

merged 7 commits into from
Oct 23, 2023

Conversation

reneepinna
Copy link
Contributor

@reneepinna reneepinna commented Oct 23, 2023

Explain changes:

  • Add PropTypes to components
  • Remove unused styles and code
  • Add responsive layouts to homepage and provide aid page

What does this PR do?

  • Feature
  • Bug fix
  • Refactor
  • Styling
  • Other:

Necessary checkmarks:

  • All Tests are Passing
  • The code will run locally

Testing Changes

  • No Tests have been changed/added
  • Some Tests have been changed/added
  • All of the Tests have been changed(Please explain)

Issues this closes

@vercel
Copy link

vercel bot commented Oct 23, 2023

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

Name Status Preview Comments Updated (UTC)
refugee-aid-fe ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2023 6:49am

@Sulton88Mehron90
Copy link
Contributor

This PR introduces several significant improvements to the codebase:

The addition of PropTypes to components enhances type checking and ensures better reliability.
I appreciate the effort to clean up the code by removing unused styles and redundant pieces. This makes the codebase leaner and more maintainable.
The responsive layouts added to the homepage and 'provide aid' page will significantly enhance the user experience across various devices.

The deployment preview from Vercel indicates that the build is successful, which is a good sign.

Great work, Renee! All the above points are confirmed, I believe this PR is ready for merging.

Thank you for all the work on this project!

@Sulton88Mehron90 Sulton88Mehron90 merged commit 4e90878 into main Oct 23, 2023
2 checks passed
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.

2 participants