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

[#30] Fill the data into the Question page #84

Merged
merged 7 commits into from
Sep 12, 2023

Conversation

manh-t
Copy link
Owner

@manh-t manh-t commented Jul 21, 2023

Close #30

What happened 👀

  • Create a MultiInputs component to add multiple inputs according to the number of answers.
  • Create an Answer component to display the corresponding components according to displayType.
  • Refactor some components to remove questionId as it's redundant.
  • Integrate the real data into the Question screen.

Insight 📝

N/A

Proof Of Work 📹

Screen.Recording.2023-09-11.at.11.00.40.mov

@manh-t manh-t added this to the 0.7.0 milestone Jul 21, 2023
@manh-t manh-t requested a review from hoangmirs July 21, 2023 08:23
@manh-t manh-t self-assigned this Jul 21, 2023
@manh-t
Copy link
Owner Author

manh-t commented Jul 21, 2023

@manh-t manh-t temporarily deployed to pull request July 21, 2023 08:25 Inactive
@manh-t
Copy link
Owner Author

manh-t commented Jul 21, 2023

Messages
📖 Coverage is a little low, take a look

Coverage in All Files

File Line Coverage Statement Coverage Function Coverage Branch Coverage
src/../MultiInputs/index.tsx (6/13) 46% (6/13) 46% (2/5) 40% (0/2) 0%
src/../TextArea/index.tsx (5/8) 63% (5/8) 63% (1/2) 50% (0/2) 0%
src/tests/error.ts (2/3) 67% (2/3) 67% (1/2) 50% (1/3) 33%
src/../Rating/index.tsx (16/20) 80% (16/20) 80% (6/6) 100% (4/13) 31%
src/../AppSlider/index.tsx (5/6) 83% (5/6) 83% (2/3) 67% (3/3) 100%
src/../Content/index.tsx (14/15) 93% (14/15) 93% (7/7) 100% (6/9) 67%
src/screens/Question/index.tsx (14/15) 93% (14/15) 93% (4/5) 80% (7/12) 58%
src/../Answer/index.tsx (17/18) 94% (17/18) 94% (3/4) 75% (10/14) 71%
src/helpers/deserializer.ts (17/18) 94% (17/18) 94% (7/7) 100% (10/12) 83%
src/../MultiChoice/index.tsx (19/20) 95% (19/20) 95% (7/8) 88% (7/8) 88%
src/../v1/requestManager.ts (19/20) 95% (19/20) 95% (6/6) 100% (3/5) 60%
src/App.tsx (3/3) 100% (3/3) 100% (1/1) 100% (0/0) NaN%
src/index.tsx (3/3) 100% (3/3) 100% (0/0) NaN% (0/0) NaN%
src/../Authentication/index.ts (4/4) 100% (4/4) 100% (2/2) 100% (0/0) NaN%
src/adapters/Base/index.ts (8/8) 100% (8/8) 100% (2/2) 100% (2/2) 100%
Other (46 more) (313/313) 100% (313/313) 100% (117/118) 99% (88/94) 94%
Total (465/487) 95% (465/487) 95% (168/178) 94% (141/179) 79%

Generated by 🚫 dangerJS against 29a92e8

@manh-t manh-t temporarily deployed to pull request July 22, 2023 02:53 Inactive
@manh-t manh-t force-pushed the feature/30-integrate-question-page branch from 7893715 to 762934a Compare July 22, 2023 03:00
@manh-t manh-t temporarily deployed to pull request July 22, 2023 03:02 Inactive
@manh-t manh-t added the @0.7.0 label Jul 27, 2023
@manh-t manh-t force-pushed the feature/31-backend-submit-survey branch 2 times, most recently from b4d8567 to da6188c Compare September 7, 2023 08:05
@manh-t manh-t force-pushed the feature/30-integrate-question-page branch 2 times, most recently from cea1c44 to 1e925a5 Compare September 7, 2023 10:29
@manh-t manh-t temporarily deployed to pull request September 7, 2023 10:30 Inactive
@manh-t manh-t temporarily deployed to pull request September 8, 2023 04:37 Inactive
@manh-t manh-t force-pushed the feature/31-backend-submit-survey branch from da6188c to 22f79e6 Compare September 8, 2023 09:47
@manh-t manh-t force-pushed the feature/30-integrate-question-page branch from ae7e948 to c68c7a7 Compare September 8, 2023 09:50
@manh-t manh-t temporarily deployed to pull request September 8, 2023 09:52 Inactive
@@ -12,7 +12,7 @@ const ElevatedButton = ({ children, isFullWidth, ...rest }: ElevatedButtonProps)
'bg-white text-black-chinese font-bold text-regular tracking-survey-tight rounded-[10px] focus:outline-none focus:shadow-outline h-14';

return (
<button type="button" className={classNames(DEFAULT_CLASS_NAMES, { 'w-full': isFullWidth })} {...rest}>
<button type="button" className={classNames(DEFAULT_CLASS_NAMES, { 'w-full': isFullWidth, 'px-8': !isFullWidth })} {...rest}>
Copy link
Owner Author

Choose a reason for hiding this comment

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

Add some padding when the button width is wrap-content

@manh-t manh-t temporarily deployed to pull request September 8, 2023 09:56 Inactive
@manh-t manh-t temporarily deployed to pull request September 11, 2023 03:43 Inactive
@manh-t manh-t force-pushed the feature/30-integrate-question-page branch from beb64a6 to 8e26642 Compare September 11, 2023 03:49
@manh-t manh-t temporarily deployed to pull request September 11, 2023 03:51 Inactive
@manh-t manh-t marked this pull request as ready for review September 11, 2023 04:02
@manh-t manh-t force-pushed the feature/30-integrate-question-page branch from 8e26642 to 40ba64a Compare September 11, 2023 04:09
@manh-t manh-t temporarily deployed to pull request September 11, 2023 04:10 Inactive
src/components/Answer/index.tsx Outdated Show resolved Hide resolved
src/components/Answer/index.tsx Outdated Show resolved Hide resolved
src/types/question.ts Show resolved Hide resolved
Base automatically changed from feature/31-backend-submit-survey to develop September 11, 2023 10:59
@manh-t manh-t temporarily deployed to pull request September 12, 2023 02:21 Inactive
@manh-t manh-t merged commit 7551322 into develop Sep 12, 2023
4 checks passed
@manh-t manh-t deleted the feature/30-integrate-question-page branch September 12, 2023 10:51
@manh-t manh-t mentioned this pull request Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Integrate] As a user, I can see the Question page
3 participants