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

Fixed hardcoded cert and added the pdf download btn #739

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

Conversation

TanmayDhobale
Copy link
Contributor

@TanmayDhobale TanmayDhobale commented May 30, 2024

PR Fixes: btw ive deleted old cert

Screenshot 2567-05-30 at 10 47 29 PM

@TanmayDhobale
Copy link
Contributor Author

@hkirat yo check out this

@AyushShivhare79
Copy link
Contributor

@TanmayDhobale This was already resolved at PR #728 for PNG
And new PDF at #738 with dynamic description.

@TanmayDhobale
Copy link
Contributor Author

@AyushShivhare79 yo i am alrdy working on this pr bro btw why u made 2 pr tho

@AyushShivhare79
Copy link
Contributor

@TanmayDhobale But he created another issue to fix dynamic description i already created the PR for that, The Thing i fixed was Typo of verified, dynamic description new PDF certificate with dynamic description, download pdf button cleaned the design of certificate and many more you can look so, can you please close this PR?

@AyushShivhare79
Copy link
Contributor

The typo of verified and description in center is still an issue in you PR although no latest certificate was added for PDF. @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

The typo of verified and description in center is still an issue in you PR although no latest certificate was added for PDF. @TanmayDhobale

fixed bro

@AyushShivhare79
Copy link
Contributor

AyushShivhare79 commented May 30, 2024

This is my first step toward open source i already fixed all the things why are you working on the solved issue? let me kick start my journey @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

This is my first step toward open source i already fixed all the things why are you working on this let me kick start my journey? @TanmayDhobale

@AyushShivhare79 lol , bro i was Already working on this issue like he told me to do but its okay .

@AyushShivhare79
Copy link
Contributor

So can you close this PR ser? @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

TanmayDhobale commented May 30, 2024

@AyushShivhare79
giphy

@AyushShivhare79
Copy link
Contributor

@TanmayDhobale I can't see he told you to fix this issue here #720

@TanmayDhobale
Copy link
Contributor Author

@TanmayDhobale I can't see he told you to fix this issue here #720

look my last marge pr

@AyushShivhare79
Copy link
Contributor

AyushShivhare79 commented May 30, 2024

He asked you for pdf one not for typo and dynamic description one ser, Also you didn't said you are working for pdf aswell there so, i fixed both before you @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

TanmayDhobale commented May 30, 2024

@AyushShivhare79
giphy

@AyushShivhare79
Copy link
Contributor

Accept it man. @TanmayDhobale Or let @hkirat Take care of this he knows the issue was already resolved.

@TanmayDhobale
Copy link
Contributor Author

Accept it man. @TanmayDhobale Or let @hkirat Take care of this he knows the issue was already resolved.

there is nothing to Accept dude

giphy

@AyushShivhare79
Copy link
Contributor

AyushShivhare79 commented May 30, 2024

I am not playing, I just solved issue and created PR 12 hrs before, I would say don't waste time work on something else I just worry why would you like to solve the solved issue men? @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

I am not playing, I just solved issue and created PR, I would say don't waste time work on something else I just worry why would you like to solve the solved issue men? @TanmayDhobale

@AyushShivhare79 bro i was Already working on this issue
giphy

@AyushShivhare79
Copy link
Contributor

Its good but he did'nt assigned it to anyone so ig i solved it first and raised PR. Check #720 @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

Its good but he did'nt assigned it to anyone so ig i solved it first and raised PR. Check #720 @TanmayDhobale

cool bro best of luck .
giphy

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.

None yet

2 participants