Skip to content
This repository has been archived by the owner on Oct 21, 2024. It is now read-only.

Migrate Rewards to Next.js #227

Open
ty730 opened this issue Jul 18, 2023 · 0 comments
Open

Migrate Rewards to Next.js #227

ty730 opened this issue Jul 18, 2023 · 0 comments
Assignees

Comments

@ty730
Copy link
Contributor

ty730 commented Jul 18, 2023

Problem

We are migrating the web-app to Next.js to take advantage of all its features to improve user experience, improve developer experience, integrate TypeScript and Inversify.js to create a more type safe and testable app, and much more.

In the next-js-migration branch we need the Rewards component to be migrated to use Next.js and TypeScript. Also, we need Unit Tests to be written to test the component.

Goal

  • Have the Rewards component in TypeScript
  • 100% coverage of Unit Tests for Rewards
  • Automate all tests using GitHub actions
  • Clean up coding style to make the codebase more consistent and readable

Additional context

Things to look for:

To get started create a next-js-migration branch in your fork of the web-app. Once you've switched to the branch then use npm install to install dependencies and npm run dev to start the development server. You'll probably need to upgrade your node version to 16.0.0 or higher to have things work correctly.

Here are links to more details about the migration including details on Next.js and TypeScript, SCSS structure and guide, naming conventions, and unit tests.

Take a look at the official Next.js documentation and official TypeScript documentation as well as this TypeScript cheat sheet.

For reference here are simple unit tests in Next.js and more complex unit tests from the old CRA framework.

@ty730 ty730 changed the title Migrate Rewards components to Next.js Migrate Rewards to Next.js Jul 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants