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

PR - fix : razorpay form in mobile view of header and addition of go to top button #135

Merged
merged 8 commits into from
Sep 3, 2024
Merged

PR - fix : razorpay form in mobile view of header and addition of go to top button #135

merged 8 commits into from
Sep 3, 2024

Conversation

sanjay7178
Copy link
Contributor

check preview : #134 (comment)

@abhisek
Copy link
Member

abhisek commented Sep 2, 2024

@sanjay7178 Can you rebase with master branch please? I see stuff that are already merged in #131

@sanjay7178 sanjay7178 force-pushed the razorpay-form-placing branch from 6c80cb2 to 7a8d835 Compare September 2, 2024 16:13
@sanjay7178
Copy link
Contributor Author

@abhisek i rebased to upstream/master to this origin pr branch , but faced few issues
as preview in #134 i wasn;t able to get css working right now as per my dev setup ie.. broke the css
image

@sanjay7178
Copy link
Contributor Author

@abhisek i rebased to upstream/master to this origin pr branch , but faced few issues as preview in #134 i wasn;t able to get css working right now as per my dev setup ie.. broke the css image

fixed in af089f6

@abhisek abhisek merged commit ae23a47 into null-open-security-community:master Sep 3, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abhisek abhisek abhisek approved these changes

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

Successfully merging this pull request may close these issues.
2 participants
@sanjay7178 @abhisek