utmmcss / mcss-website-frontend

MCSS Website Frontend
https://mcss.club
MIT License
0 stars 1 forks source link

MCSS-90: Transition to new cms endpoints #93

Closed eruditeme closed 11 months ago

eruditeme commented 11 months ago

What's Inside

... full details of acceptance criteria documented in the linked GitHub issue

netlify[bot] commented 11 months ago

Deploy Preview for mcss-website ready!

Name Link
Latest commit ad7e1f96134c5b0321777131aef34e154150e86c
Latest deploy log https://app.netlify.com/sites/mcss-website/deploys/653c74f0a833610007d67923
Deploy Preview https://deploy-preview-93--mcss-website.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

Lighthouse
1 paths audited
Performance: 95
Accessibility: 84
Best Practices: 100
SEO: 86
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

eruditeme commented 11 months ago

Currently getting 403 forbidden errors from backend. Will look into the fix tmw

eruditeme commented 11 months ago

Also I can see a blog rn from preview, is that different than what you were mentioning?

Screen Shot 2023-10-24 at 11 58 22 PM
anthonytedja commented 11 months ago

Also I can see a blog rn from preview, is that different than what you were mentioning?

Screen Shot 2023-10-24 at 11 58 22 PM

yeah that seems to be the same data structure but if u click on it youre not gonna see the event details text, same with the team stuff with the links. it just so happens that some stuff is the same as the old cms so it shows

eruditeme commented 11 months ago

Finished with the team members and events but can't seem to get blogs to show up. In blogSlice.ts, "here" prints but "there" doesnt. Not too sure what the issue is

Screen Shot 2023-10-25 at 8 17 12 PM
anthonytedja commented 11 months ago

Finished with the team members and events but can't seem to get blogs to show up. In blogSlice.ts, "here" prints but "there" doesnt. Not too sure what the issue is Screen Shot 2023-10-25 at 8 17 12 PM

not sure why, if it was printing the object but its not returning the intended value it probably got updated before printing https://stackoverflow.com/questions/22306279/is-it-possible-to-flush-the-console-make-it-print-immediately but its just a string so no idea, we can take a look at your code if you push your changes here

eruditeme commented 11 months ago

I just pushed what I have so far in my code

eruditeme commented 11 months ago

Also I found the issue. The cover on the backend allows multiple images. I'm not too sure how to handle the ${PROCESS.ENV} part for multiple images, maybe an array that stores each image? I believe a carousel would also need to b added to blogs if there is multiple images.

eruditeme commented 11 months ago

The frontend is fetching from find instead of findOne. In the backend, find deletes content for blogs and events

eruditeme commented 11 months ago

We would need to add findOne to frontend as that includes content but not sure how to adjust tht in next.js

Screen Shot 2023-10-27 at 7 01 07 PM
anthonytedja commented 11 months ago

The frontend is fetching from find instead of findOne. In the backend, find deletes content for blogs and events

ah ic, forgot when we decided this but ok lets call it out of scope for this ticket then