Web-Dev-Path / web-dev-path

The Web Dev Path platform. Progressive Web App (PWA). Next.js rules!
https://webdevpath.co
GNU General Public License v3.0
33 stars 11 forks source link

Adjust the `<Image />` components to meet Next.js 13 requirements #148

Closed mariana-caldas closed 1 year ago

mariana-caldas commented 1 year ago

What do we need to build or fix?

In order to update to Next.js 13, we are using import Image from 'next/legacy/image' on the components Card.js and TwoColumns.js. We need to adjust the <Image /> components on those components so they match the new Next.js 13 requirements and we can, then, remove the legacy flag.

Technical details

Adapt the <Images /> components by checking the documentation guidelines

Approach suggestions

Deadline

Please keep in mind that once you assign this task to yourself, you'll need to complete it in 10 days.

Acceptance criteria

Faith-Nchifor commented 1 year ago

App still functions as expected

mariana-caldas commented 1 year ago

App still functions as expected

I'm not sure what you mean, @Faith-Nchifor . Feel free to elaborate on your comment and maybe questions and ping me or @cherylli on Slack if necessary.

Faith-Nchifor commented 1 year ago

I'm sorry for not explaining clearly @mariana-caldas. I meant the changes for the components are okay.
Personally, I tried to make the changes before assigning myself to this task but I got some errors. I know see what @cherylli did and hopefully I will do the next task successfully.