Skip to content

Fix cards on About page #78 #96

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

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

Conversation

Arthur-Maybe
Copy link
Contributor

Change the carousel at the bottom of the About page (in the Our Impact) section to match what's shown in the Figma.

Change the carousel at the bottom of the About page (in the Our Impact) section to match what's shown in the Figma.
@nivedhanatarajan7 nivedhanatarajan7 self-requested a review October 12, 2024 19:05
@nivedhanatarajan7
Copy link
Collaborator

Please fix the issues flagged by the merge check to ensure that the CI and code style check is passing. Thank you!

@jayp822
Copy link
Contributor

jayp822 commented Oct 20, 2024

@Arthur-Maybe hey man your code looks good all you have to do it run npm run fl fl means format and lint its a custom command I made to make it easier to format your code and lint it before you commit. If you get alot of red then its because the linter rejected it. Your code got failed checks because of just an enter button. Just run the command above and recommit and push. If you get red and its from a file you didnt touch then it means someone else f'ed up and push bad code to the main branch
Screenshot 2024-10-20 at 7 38 59 PM

@nivedhanatarajan7
Copy link
Collaborator

Hi! You might need to sync the files, as there is a merge check error that might be related to some of the code style issues that were resolved in some future pull requests. Thank you!

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.

3 participants