Conversation
There was a problem hiding this comment.
Hey really appreciate the attention to detail when making the certificate modular, thank you for that.
One question: is the stuff under /api dead code? Doesn't seem to be referenced by anything. Other than that, great code, great design, feel free to merge! I saw your message in the tech chat about creating a separate project member collection btw and I totally agree, feel free to merge this in the meantime tho and we can remove the script etc when it comes to it.
|
I think i will be removing that soon, the idea was to generate an image as og-image when a person paste their certificate link so that it would be displayed under say linkedin or discord with their name on it. it added a lot of unnecessary complexity and im not convinced that it is easy to maintain as more styles are added it will probably be replaced by just a general static image saying "certificate of participation 2025" that someone can draft up on figma and place into the website? |
|
Aye sounds good 👍 |
Improve Build Script (GitHub Actions workflow)
Revert this commit when the event's over
|
I'm not sure if the all certificate page makes sense - wouldn't it be better to take you to the user's certificate when you click on their icon in their project page? The rest is good, I'll leave it to you if you wanna merge now or wait for the project member collection thingy tho |
Refactor About Page
…ckathon.com/" This reverts commit a91fd3d.
🚧 Sponsor page spike
Refactor payload fetch/parse and remove most fallback data
…ebsite-v4 into wdcc-certificates
5 participants
Description
a digital certificate??? 🫣
How To Review
all file under
src/app/(main)/certificate/view/[uuid]CertificateCollection.tsx