Skip to content
This repository was archived by the owner on Dec 10, 2024. It is now read-only.

[DEV-050] add wizard form#70

Open
AipioxTechson wants to merge 3 commits into
developfrom
DEV-050
Open

[DEV-050] add wizard form#70
AipioxTechson wants to merge 3 commits into
developfrom
DEV-050

Conversation

@AipioxTechson

Copy link
Copy Markdown
Member

[DEV-050] Wizard Form

Fixes: #50

  • Has the code been formatted?
  • Have dependencies/packages been added?

Changes

Explain the changes made at a higher level

Testing

  • Does the code have sufficient automated test coverage? If not, include a reasoning and steps required to manually QA
  • No because this is data orientated as such can't be mocked with component tests

@AipioxTechson AipioxTechson requested a review from jcserv January 25, 2021 04:51
@vercel

vercel Bot commented Jan 25, 2021

Copy link
Copy Markdown

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/white-van/dialog/r004vtub2
✅ Preview: https://dialog-git-dev-050.white-van.vercel.app

@jcserv jcserv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Could you write some tests for it?

Comment thread frontend/pages/index.tsx
export default connect(null, {
displaySuccessSnack,
})(Home);
export default Home;

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oooh TIL

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Wizard Form

2 participants