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.

Sign up for GitHub

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

Jump to bottom

[wip] [#18] [Backend] As a logged-in user, I can see the Survey Detail screen #76

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from
Draft

[wip] [#18] [Backend] As a logged-in user, I can see the Survey Detail screen #76

wants to merge 1 commit into from

Conversation

Thieurom
Copy link
Collaborator

@Thieurom Thieurom commented Sep 7, 2023

What happened 👀

Describe the big picture of your changes here to communicate to the team why we should accept this pull request.

Insight 📝

Describe in details how to test the changes, which solution you tried but did not go with, referenced documentation is
welcome as well.

Proof Of Work 📹

Show us the implementation: screenshots, gif, etc.

@Thieurom Thieurom added this to the 0.6.0 milestone Sep 7, 2023
@Thieurom Thieurom self-assigned this Sep 7, 2023
@codecov
Copy link

codecov bot commented Sep 7, 2023 •
edited
Loading

Codecov Report

Patch coverage has no change and project coverage change: -4.85% ⚠️

Comparison is base (7f19920) 49.49% compared to head (09276ad) 44.64%.
Report is 1 commits behind head on develop.

Additional details and impacted files 
@@             Coverage Diff             @@
##           develop      #76      +/-   ##
===========================================
- Coverage    49.49%   44.64%   -4.85%     
===========================================
  Files           36       44       +8     
  Lines          396      439      +43     
===========================================
  Hits           196      196              
- Misses         200      243      +43
Flag Coverage Δ
unittests 44.64% <0.00%> (-4.85%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed Coverage Δ
lib/model/answer_model.dart 0.00% <0.00%> (ø)
lib/model/question_model.dart 0.00% <0.00%> (ø)
lib/model/response/answer_response.dart 0.00% <0.00%> (ø)
lib/model/response/question_response.dart 0.00% <0.00%> (ø)
...ib/model/response/survey_detail_data_response.dart 0.00% <0.00%> (ø)
lib/model/response/survey_detail_response.dart 0.00% <0.00%> (ø)
lib/model/survey_detail_model.dart 0.00% <0.00%> (ø)
lib/repositories/survey_repository.dart 41.17% <0.00%> (-17.16%) ⬇️
lib/utils/string_extension.dart 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@nkhanh44 nkhanh44 Awaiting requested review from nkhanh44 nkhanh44 will be requested when the pull request is marked ready for review nkhanh44 is a code owner

@minhnimble minhnimble Awaiting requested review from minhnimble minhnimble will be requested when the pull request is marked ready for review minhnimble is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

@Thieurom Thieurom

Labels
type : feature
Projects
None yet
Milestone
0.6.0
Development

Successfully merging this pull request may close these issues.

[Backend] As a logged-in user, I can see the Survey Detail screen
1 participant
@Thieurom