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

BACKOFFICE FE/UI/Labeling Tool: Update UI/UX handle qc_status #11

Merged
merged 23 commits into from
Sep 28, 2022

Conversation

cuongho2
Copy link

Pre-flight checklist

  • Unit tests for all non-trivial changes
  • Tested locally
  • Updated wiki

@vercel
Copy link

vercel bot commented Aug 24, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
make-sense ✅ Ready (Inspect) Visit Preview Sep 28, 2022 at 1:24AM (UTC)

Copy link

@choipd choipd left a comment

Choose a reason for hiding this comment

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

[major]

  1. There are many files changed. This makes code review hard. Please separate PR by intention.
  2. Please describe the detail what you trying to change in PR description
  3. Please try not to change the previous code format. This makes many changes which are not related this PR.

package.json Outdated Show resolved Hide resolved
src/index.tsx Outdated Show resolved Hide resolved
Copy link
Author

@cuongho2 cuongho2 left a comment

Choose a reason for hiding this comment

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

I was resolved all issue, please check it again, thank.

src/views/PopupView/LoginPopup/LoginPopup.tsx Outdated Show resolved Hide resolved
@cuongho2 cuongho2 changed the title Sn 568 v1 BACKOFFICE FE/UI/Labeling Tool: Update UI/UX handle qc_status Sep 5, 2022
@cuongho2
Copy link
Author

cuongho2 commented Sep 5, 2022

[major]

  1. There are many files changed. This makes code review hard. Please separate PR by intention.
  2. Please describe the detail what you trying to change in PR description
  3. Please try not to change the previous code format. This makes many changes which are not related this PR.

@choipd, I will remember and description indent task detail more than, thank for you advise.

@cuongho2
Copy link
Author

cuongho2 commented Sep 5, 2022

@choipd , can you approve my new change code.

@cuongho2 cuongho2 requested a review from choipd September 6, 2022 01:19
import {CSSHelper} from '../../../logic/helpers/CSSHelper';
import {APIService} from '../../../services/API';
import {PopupWindowType} from '../../../data/enums/PopupWindowType';
import { Input } from '@material-ui/core';
Copy link

@choipd choipd Sep 6, 2022

Choose a reason for hiding this comment

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

[Minor]
Please keep the previous code format. It would be nice to focus on which part of the codes changed.

Copy link
Author

Choose a reason for hiding this comment

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

@choipd, I was change code to previous in this view.

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.

3 participants