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.

Sign up for GitHub

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

Jump to bottom

Added scroll to top button #112

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Added scroll to top button #112

wants to merge 1 commit into from

Conversation

meghanakn473
Copy link

i have added the feature top to scroll button.
please review and merge and assign the labels kindly
@tmsagarofficial @BamaCharanChhandogi

BamaCharanChhandogi
BamaCharanChhandogi reviewed Nov 6, 2024
View reviewed changes
Copy link
Owner

@BamaCharanChhandogi BamaCharanChhandogi 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 for your contribution! Your pull request has been submitted successfully. A maintainer from GitFinder team will review it as soon as possible. We appreciate your support in making this project better

@meghanakn473
Copy link
Author

@tmsagarofficial and @BamaCharanChhandogi please review and merge....in concern of leaderboard ending soon

BamaCharanChhandogi
BamaCharanChhandogi requested changes Nov 7, 2024
View reviewed changes
Copy link
Owner

@BamaCharanChhandogi BamaCharanChhandogi left a comment

Choose a reason for hiding this comment

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

I can not see the topToScroll button. Please provide a screenshot or something.

@meghanakn473
Copy link
Author

@BamaCharanChhandogi can u please provide .env file details so that I can run and provide the screenshot

@BamaCharanChhandogi
Copy link
Owner

@BamaCharanChhandogi can u please provide .env file details so that I can run and provide the screenshot

You have to set up Firebase and GitHub key. You will have all the .env details.

@meghanakn473
Copy link
Author

@BamaCharanChhandogi can u please provide .env file details so that I can run and provide the screenshot

You have to set up Firebase and GitHub key. You will have all the .env details.

i have set it @BamaCharanChhandogi but im facing some issues
wen the page is opened,, and wen i click on lets go button its not navigating to next page ,,,,how do i correct this

@BamaCharanChhandogi
Copy link
Owner

@BamaCharanChhandogi can u please provide .env file details so that I can run and provide the screenshot

You have to set up Firebase and GitHub key. You will have all the .env details.

i have set it @BamaCharanChhandogi but im facing some issues wen the page is opened,, and wen i click on lets go button its not navigating to next page ,,,,how do i correct this

I have to see the error.

meghanakn473 added a commit to meghanakn473/Open-source-Practice that referenced this pull request Nov 9, 2024
@meghanakn473
Create Meghana-pr.md
bfa6379 
hi gssoc team
these are my 4 pr's that has not been merged

level 3
ajaynegi45/Uttarakhand-Culture-NewUI#256

level 3
tushargupta1504/Medical-Website#642

level 3
Mohit5Upadhyay/WeatherApplication#19

level 2
BamaCharanChhandogi/GitFinder#112

please look and allow merge
@meghanakn473 meghanakn473 mentioned this pull request Nov 9, 2024
Closed
@meghanakn473
Copy link
Author

@BamaCharanChhandogi can u please provide .env file details so that I can run and provide the screenshot

You have to set up Firebase and GitHub key. You will have all the .env details.

i have set it @BamaCharanChhandogi but im facing some issues wen the page is opened,, and wen i click on lets go button its not navigating to next page ,,,,how do i correct this

I have to see the error.

image
@BamaCharanChhandogi

@meghanakn473
Copy link
Author

@BamaCharanChhandogi did u go through the error? What shd I do further?

meghanakn473 added a commit to meghanakn473/Open-source-Practice that referenced this pull request Nov 10, 2024
@meghanakn473
Create meghana-pr.md
66b1657 
hi gssoc team,
i had made 2 pr's below, there is no response for the pr i made, kindly check and assign the points

level 3
shivamyadavrgipt/Social_media_management#230 (comment)

level 3
BamaCharanChhandogi/GitFinder#112 (comment)
@meghanakn473 meghanakn473 mentioned this pull request Nov 10, 2024
Merged
@aslams2020
Copy link

@BamaCharanChhandogi Can Please review this PR on your Own. Please do it before evening.
As this PR seems to have some required changes issues -- I prefer you review it @BamaCharanChhandogi

@meghanakn473 If before evening, it's not reviewed then create PR for it.

meghanakn473 added a commit to meghanakn473/Open-source-Practice that referenced this pull request Nov 10, 2024
@meghanakn473
Create meghanakn-pr.md
7a2d549 
hi gssoc team,
the below pr's has not been merged

level 2
shivamyadavrgipt/Social_media_management#231

the below sanjay-kv#112 pr admin as not responded yet, 
level 2
BamaCharanChhandogi/GitFinder#112

shivamyadavrgipt/Social_media_management#230 this pr was assigned at level 3 but while merging u assigned only level 2

kindly check and assign the points
thank you :)
@meghanakn473 meghanakn473 mentioned this pull request Nov 10, 2024
Merged
@aslams2020
Copy link

merged - sanjay-kv/Open-source-Practice#1526

BamaCharanChhandogi
BamaCharanChhandogi reviewed Nov 10, 2024
View reviewed changes
Copy link
Owner

@BamaCharanChhandogi BamaCharanChhandogi left a comment

Choose a reason for hiding this comment

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

The code is wrong. I can not merge.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BamaCharanChhandogi BamaCharanChhandogi BamaCharanChhandogi requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@meghanakn473 @BamaCharanChhandogi @aslams2020 @meghanakn22