Skip to content

feature/react ojt final project#5

Open
Aung-Thu-Hein wants to merge 9 commits into
mainfrom
feature/react-ojt-final-project
Open

feature/react ojt final project#5
Aung-Thu-Hein wants to merge 9 commits into
mainfrom
feature/react-ojt-final-project

Conversation

@Aung-Thu-Hein

Copy link
Copy Markdown
Owner

<< Changes >>

  • validate form value
  • add to local storage
  • dipslay user information in table using react router

final_pj2
final project

@yewai01
yewai01 requested review from yewai01 and removed request for yewai01 December 30, 2022 04:46
Comment thread package.json Outdated
"@testing-library/react": "^13.4.0",
"@testing-library/user-event": "^13.5.0",
"bootstrap": "^5.2.3",
"formik": "^2.2.9",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ma use tae dependency ko delete pay pr

Comment thread src/components/Birthday.js Outdated

const Birthday = (props) => {

console.log("Props" ,props)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

console.log twe delete pay pr


const schema = Yup.object().shape({

userName: Yup

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

space a po twe delete pay pr

Comment thread src/components/FormComponent.js Outdated
if (data) {
data = JSON.parse(data);
if (data.length > 9) {
setValue('userId', 'ID-00' + `${data.length + 1}`)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

prefix zero ko hard code ma yay pr nae
try this
let n= 10
String(n).padStart(4, '0') // 0010

Comment thread src/hooks/useLocalStorage.js Outdated
@@ -0,0 +1,21 @@
import { useEffect, useState } from "react";

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

d hook ka ma use yin delete lox ya pr tl
pee tot javascript file twe ko space 2 nae pl setting cha ht pay pr aku ka space 4 twe pyit nay pr tl all page

@@ -0,0 +1,235 @@
import Button from 'react-bootstrap/Button';

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

user role mhr validation sit pay pr

<Col sm={2}><Form.Label>Gender</Form.Label></Col>
<Col sm={1}>
<Form.Check
{...register('gender')}

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

radio button mhr id nae chate pay pr

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants