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 the cert design !! #609

Merged
merged 6 commits into from
May 27, 2024
Merged

fixed the cert design !! #609

merged 6 commits into from
May 27, 2024

Conversation

TanmayDhobale
Copy link
Contributor

PR Fixes:

@hkirat
/certificate not working on app.100x and on local also and not able to see certificate so change logic in cert.ts and make false and its worked !!! check that pls ( const courses = purchases.filter((x) => x.certIssued === false); )

Resolves #596

Checklist before requesting a review

  • I have performed a self-review of my code
  • I assure there is no similar/duplicate pull request regarding same issue

Screenshot 2567-05-13 at 4 53 05 PM

@TanmayDhobale
Copy link
Contributor Author

for an seed file pls check this pr #599

@TanmayDhobale
Copy link
Contributor Author

TanmayDhobale commented May 13, 2024

btw its give 5 certificate idk why maybe bcs of an 5 course we have
and i think don`t so we need to save the data in db bcs we have an fix date like 3 mont cohort

@piyushhsainii
Copy link

NO HELP CREDIT? 😣 @TanmayDhobale

@TanmayDhobale
Copy link
Contributor Author

TanmayDhobale commented May 13, 2024

NO HELP CREDIT? 😣 @TanmayDhobale
Special thanks to @piyushhsainii for wasting time, I'm kidding really thanks bro for help

@yntpdotme
Copy link

@TanmayDhobale Can we do it in Dark Mode...!!

@hkirat
Copy link
Contributor

hkirat commented May 27, 2024

/bounty $50
I think we need to do the same thing for the pdf as well

@hkirat hkirat merged commit a3b5068 into code100x:main May 27, 2024
@TanmayDhobale
Copy link
Contributor Author

/bounty $50 I think we need to do the same thing for the pdf as well

umm pdf ???

@TanmayDhobale
Copy link
Contributor Author

/bounty $50 I think we need to do the same thing for the pdf as well

umm pdf ??? @hkirat

@hkirat
Copy link
Contributor

hkirat commented May 27, 2024

You can download pdf as well

@hkirat
Copy link
Contributor

hkirat commented May 27, 2024

That design is still the old one

@TanmayDhobale
Copy link
Contributor Author

TanmayDhobale commented May 27, 2024

@hkirat i think ervyone have cert access rn but the thing is 1 to 100 still going on

update

@TanmayDhobale
Copy link
Contributor Author

TanmayDhobale commented May 27, 2024

Screenshot 2567-05-28 at 12 58 37 AM
@hkirat why its showing wrong

@yntpdotme
Copy link

yntpdotme commented May 28, 2024

@TanmayDhobale Can we update the 100xdevs

  • To : 100xDevs
  • Font-Face : Be Vietnam Pro
  • Weight : Semi-Bold

@AyushShivhare79
Copy link
Contributor

@TanmayDhobale I fixed this issue of showing the wrong course #728

And @hkirat Solved the pdf issue as well on #738

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.

bug: revert to this type of cert
5 participants