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

Migrate all next/legacy/image imports to next/image #899

Closed
3 tasks
eleanorreem opened this issue May 10, 2024 · 4 comments
Closed
3 tasks

Migrate all next/legacy/image imports to next/image #899

eleanorreem opened this issue May 10, 2024 · 4 comments
Assignees
Labels
complexity: beginner Time needed to do this ticket is less than 1 day. dependencies Updates a dependency file maintenance Maintenance / chore work priority: 1+week Should be prioritized next week or longer. state: approved Ready to go. Not blocked or pending.

Comments

@eleanorreem
Copy link
Contributor

Overview

When we upgraded nextjs, we didn't migrate to the new next/image imports. This ticket is tidying up the legacy imports.

Action Items

  • Go through the repo and replace all instances of next/legacy/image with new import of next/image.
  • Check each instance and ensure it has not broken the image in UI
  • Run Cypress tests

Resources/Instructions

See https://nextjs.org/docs/app/building-your-application/upgrading/app-router-migration section on next/image migration.

@eleanorreem eleanorreem added dependencies Updates a dependency file complexity: beginner Time needed to do this ticket is less than 1 day. maintenance Maintenance / chore work state: approved Ready to go. Not blocked or pending. priority: 1+week Should be prioritized next week or longer. labels May 10, 2024
@anmol-fzr
Copy link
Contributor

@eleanorreem I would like to work on this issue, please assign this to me.

@LouisVdwl
Copy link

Hey @eleanorreem, I would be happy to work on it ! Can you assign me the issue ? 😄

@kyleecodes
Copy link
Member

Hey, thank you for your patience. @anmol-fzr I have assigned you the issue. If you would like to work on something else instead we'll assign someone else.
@LouisVdwl thank you for your interest! We have more beginner friendly issues being published soon so check back for similar issues.

Copy link
Contributor

Thank you for your interest in contributing to Chayn! Please carefully read the CONTRIBUTING.md file and the README.md file for guidance. Let us know if you have any questions. Good luck!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
complexity: beginner Time needed to do this ticket is less than 1 day. dependencies Updates a dependency file maintenance Maintenance / chore work priority: 1+week Should be prioritized next week or longer. state: approved Ready to go. Not blocked or pending.
Projects
None yet
Development

No branches or pull requests

4 participants