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

Feat/contributor page #237

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

Conversation

Sawan-Kushwah
Copy link

issue #234

Uttarakhand.Culture.-.Explore.Devbhoomis.Heritage.Traditions.and.Natural.Beauty.Mozilla.Firefox.2024-10-29.22-25-55.1.mp4

Here we you can see the real time data of contributor !!
Available in both light and dark theme !!
if UI changes need comment below i will do it

Copy link

vercel bot commented Oct 29, 2024

@Sawan-Kushwah is attempting to deploy a commit to the ajaynegi45's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

🎉 Thank you @Sawan-Kushwah for your contribution! Your pull request has been submitted successfully, and a maintainer will review it soon. We’re excited to have you on board! For updates and community support, join our Discord and be part of the conversation. Remember to star the repo 🌟 to help us grow, and follow @ajaynegi45 to stay in the loop and increase the visibility of your contributions!

@ajaynegi45
Copy link
Owner

authorize it

Kindly follow these rules #237 (review)

Copy link

vercel bot commented Oct 30, 2024

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

Name Status Preview Comments Updated (UTC)
uttarakhand-culture ❌ Failed (Inspect) Nov 12, 2024 5:33pm

@Sawan-Kushwah
Copy link
Author

@ajaynegi45 which rule ?

@ajaynegi45
Copy link
Owner

🎉 Thank you @Sawan-Kushwah for your contribution! Your pull request has been submitted successfully, and a maintainer will review it soon. We’re excited to have you on board! For updates and community support, join our Discord and be part of the conversation. Remember to star the repo 🌟 to help us grow, and follow @ajaynegi45 to stay in the loop and increase the visibility of your contributions!

This

@Sawan-Kushwah
Copy link
Author

Sawan-Kushwah commented Oct 30, 2024

@ajaynegi45 I DONT have any discord , but i can star the repo!
i have given star to this repo

Can i get level 3 for this PR

@Sawan-Kushwah
Copy link
Author

image
@ajaynegi45

@ajaynegi45
Copy link
Owner

i have given star to this repo

Also follow

@Sawan-Kushwah
Copy link
Author

@ajaynegi45 done

@ajaynegi45
Copy link
Owner

Sawan-Kushwah

For getting level 3. Improve the feature Functionality.

@Sawan-Kushwah
Copy link
Author

@ajaynegi45 can you please what specific feature want ?

@ajaynegi45
Copy link
Owner

@ajaynegi45 can you please what specific feature want ?

I don't know, but for level 3 You have to add more code according to GSSOC. You can Improve UI or anything

@Sawan-Kushwah
Copy link
Author

@ajaynegi45
i have added all the things for a contribution page
Added animation
Available for both light and dark theme
User details along with there profile link
repo status
smooth hover effects

this is enough for an level 3 issue as in other repo they have given me level 3

@ajaynegi45
Copy link
Owner

Visit Preview

Hover animation is slow. Make it little bit faster

@ajaynegi45 ajaynegi45 added enhancement New feature or request status: ready for dev You can asked for this issue to be assigned (if not already assigned) hacktoberfest hacktoberfest gssoc GirlScript Summer Of Code hacktoberfest-accepted hacktoberfest level3 GirlScript Summer Of Code - 35 points gssoc-ext GirlScript Summer Of Code labels Oct 30, 2024
@Sawan-Kushwah
Copy link
Author

@ajaynegi45 done !!

@ajaynegi45
Copy link
Owner

@ajaynegi45 i have added all the things for a contribution page Added animation Available for both light and dark theme User details along with there profile link repo status smooth hover effects

this is enough for an level 3 issue as in other repo they have given me level 3

If anyone wants to contribute, you can show them open issues.

On click contributor profile redirect to new page where you can show what they contribute to this project.

@Sawan-Kushwah
Copy link
Author

@ajaynegi45
image

also added link to contributor to show them there contribution

on click to open issue it will show all issue
on click to contributor it will show all contributor

Copy link
Owner

@ajaynegi45 ajaynegi45 left a comment

Choose a reason for hiding this comment

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

Hi @Sawan-Kushwah,

Thank you for your prompt work on this issue. I appreciate your dedication to the project.

I have reviewed your work, and I need some changes.

  • Pagination is not working properly

    Screen.Recording.2024-11-01.at.9.21.35.AM.mov

Once again, thank you for your contribution! ❤️

@Sawan-Kushwah
Copy link
Author

@ajaynegi45 check it once
now pagination is working !!!

@Sawan-Kushwah
Copy link
Author

@ajaynegi45 WILL YOU PLEASE MERGE IT NOW ?

Copy link
Owner

@ajaynegi45 ajaynegi45 left a comment

Choose a reason for hiding this comment

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

Hi @Sawan-Kushwah,

Thank you for your prompt work on this issue. I appreciate your dedication to the project.

I have reviewed your work, and I need some changes.

  • Container are not equal in open issues and pagination is not working.

    Screenshot 2024-11-01 at 9 07 45 PM
  • Also show star and forks list with proper paginationa

Once again, thank you for your contribution! ❤️

@Sawan-Kushwah
Copy link
Author

@ajaynegi45
Screenshot 2024-11-02 153538
i have fix its styling !!

@Sawan-Kushwah
Copy link
Author

@ajaynegi45 hey please response and merge this PR !!!
else it is of no use

@Sawan-Kushwah
Copy link
Author

@MastanSayyad please merge my PR else i won't any points

Copy link
Owner

@ajaynegi45 ajaynegi45 left a comment

Choose a reason for hiding this comment

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

Hi @Sawan-Kushwah,

Thank you for your prompt work on this issue. I appreciate your dedication to the project.

I have reviewed your work, and I need some changes.

at JSON.parse ()
at O (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:94083) {
digest: '754964660'
}
Error occurred prerendering page "/auth". Read more: https://nextjs.org/docs/messages/prerender-error
ReferenceError: window is not defined
at 3602 (/vercel/path0/.next/server/app/auth/page.js:1:2572)
at t (/vercel/path0/.next/server/webpack-runtime.js:1:143)
at 9325 (/vercel/path0/.next/server/app/auth/page.js:1:133339)
at Object.t [as require] (/vercel/path0/.next/server/webpack-runtime.js:1:143)
at require (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:16:18490)
at I (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:94362)
at /vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:96668
at F._fromJSON (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:97106)
at JSON.parse ()
at O (/vercel/path0/node_modules/next/dist/compiled/next-server/app-page.runtime.prod.js:12:94083)
Generating static pages (12/17)
✓ Generating static pages (17/17)
Export encountered errors on following paths:
/auth/page: /auth
Error: Command "npm run build" exited with 1

Once again, thank you for your contribution! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gssoc GirlScript Summer Of Code gssoc-ext GirlScript Summer Of Code hacktoberfest hacktoberfest hacktoberfest-accepted hacktoberfest level3 GirlScript Summer Of Code - 35 points status: ready for dev You can asked for this issue to be assigned (if not already assigned)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants