Skip to content

Conversation

AfolabiOlajide
Copy link
Collaborator

Summary:

Removed modal repetitions and optimized code for filter dropdown

Fixes Issue/Linear Ticket

This could be an existing issue or a linear ticket

  • Github Issue Example: My PR Closes #{ISSUE}
  • Linear Ticket Example: Fixes ID-#{ISSUE}

Changes proposed

Talk about the things you did eg. files changes, dependencies installed e.t.c

Check List (Check all the applicable boxes)

🚨Please review the [style guide for contributing](add link here) and [guidelines for contributing](add link here) to this repository.

  • My code follows the code style of this project.
  • This PR does not contain plagiarized content.
  • The title and description of the PR is clear and explains the approach.
  • I am making a pull request against the main branch (left side).
  • My commit messages styles matches our requested structure.
  • My code additions will fail neither code linting checks nor unit test.
  • I am only making changes to files I was requested to.

Screenshots/ Videos

Filter.Dropdown.Optimized-1.mp4

@vercel
Copy link

vercel bot commented Oct 22, 2023

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

Name Status Preview Comments Updated (UTC)
zuriportfolio-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2023 9:58pm
zuriportfolio-frontend-pw1h ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 22, 2023 9:58pm
zuriportfolio-frontend1 ❌ Failed (Inspect) Oct 22, 2023 9:58pm

@vercel
Copy link

vercel bot commented Oct 22, 2023

Deployment failed with the following error:

Resource is limited - try again in 1 minute (more than 100, code: "api-deployments-free-per-day").

Copy link
Collaborator

@dev-bazz dev-bazz left a comment

Choose a reason for hiding this comment

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

hmm..

Copy link
Collaborator

@dev-bazz dev-bazz left a comment

Choose a reason for hiding this comment

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

wat

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