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

feat(frontend): add discard changes button on edit wheel screen bb-467 #601

Open
wants to merge 126 commits into
base: main
Choose a base branch
from
Open

feat(frontend): add discard changes button on edit wheel screen bb-467 #601

wants to merge 126 commits into from

Conversation

iqbalalayubbi
Copy link
Collaborator

Solved Issue

  • when user make some changes on edit manual wheel part the discard changes button active. Then, if user click this button again it can be make inactive again
  • when user make some changes and switch into retake quiz section. It can be reset the value of balance wheel part.

Demo

fix-discard-changes-button.mp4

iqbalalayubbi and others added 30 commits August 18, 2024 21:49
@iqbalalayubbi
feat: slicing form design ui bb-9
a6a4d4a
@iqbalalayubbi
Merge branch '9-feat-sign-up' into fix-lint
6789363
@iqbalalayubbi
fix: remove lint css root folder bb-0
1a07082
@iqbalalayubbi
feat: make toggle visibility password bb-9
0d9786c
@iqbalalayubbi
feat: add user data in redux store bb-9
91c4a09
@iqbalalayubbi
feat: make validation confirm password bb-9
eef6fb3
@iqbalalayubbi
fix: refactor unused code and comment bb-9
4f1c228
@iqbalalayubbi
style: adjust css variabel to match with other bb-9
5fc8a04
@iqbalalayubbi
refactor: make other props instead className bb-9
fcffd3e
@iqbalalayubbi
style: adjust css variabel conflict bb-9
f6aaac0
@iqbalalayubbi
Merge branch 'main' into 9-feat-sign-up
5842e98
@iqbalalayubbi
Merge branch 'main' into 9-feat-sign-up
8c9ae6e
@iqbalalayubbi
Merge branch 'main' into 9-feat-sign-up
18bc6f6
@github-actions @iqbalalayubbi
chore: release-1.6.0 (#95)
d97f37c 
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@iqbalalayubbi
fix: handle conflict with main branch bb-9
dc53cac
@iqbalalayubbi
fix: update version in package.json bb-9
2e7c7b7
@iqbalalayubbi
refactor: using getValidClassNames and seperate each style in compone…
a1bfc0e 
…nt bb-9
@iqbalalayubbi
fix: handle conflict in app.tsx bb-9
979b4e6
@iqbalalayubbi
Merge branch 'main' into 9-feat-sign-up
24512fe
@iqbalalayubbi
style: change all changelog file bb-9
3451234
@iqbalalayubbi
refactor: remove unused img and code bb-9
ef12676
@iqbalalayubbi
style: change folder structure and handle typo name in class bb-9
e985ddc
@iqbalalayubbi
fix: update package.json bb-9
ba27cbc
@iqbalalayubbi
fix: update with main branch bb-9
78a277c
@iqbalalayubbi
fix: solve UI issue in sign up UI bb-9
21a4f00
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
0967b82 
…4-bebalance into 9-feat-sign-up
@iqbalalayubbi
feat: redirect user to root after sign up bb-9
e290950
@iqbalalayubbi
feat: navigate user to root path bb-9
37dc177
@iqbalalayubbi
refactor: remove use navigate in sign up form bb-9
078b9c9
@iqbalalayubbi
fix: solve conflict with main branch bb-9
69db1ae
iqbalalayubbi and others added 16 commits September 22, 2024 21:08
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
55bc3df 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
95a5ff6 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
refactor: change slider ref into prop value bb-467
cfc9074
@iqbalalayubbi
refactor: replace timing function with use effect bb-467
2c79dd5
@iqbalalayubbi
refactor: remove areChangesDiscarded prop from slider component bb-467
a390ccd
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
990ba36 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
refactor: using state hook to replace ref hook for scores data bb-467
433fce2
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
74ed482 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
refactor: call api to get original scores bb-467
f9bae8e
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
65034ea 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@fshabanov
Merge branch 'main' into 467-feat-add-discard-changes-button-on-edit-…
60010f6 
…wheel-screen
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
83b0ed6 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
8c1d9da 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
fix: discard button disabled when click and reset the scores when swi…
07f73c3 
…tch to retake quiz bb-467
@iqbalalayubbi
Merge branch 'main' of https://github.com/BinaryStudioAcademy/bsa-202…
3fdb953 
…4-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi
Merge branch '467-feat-add-discard-changes-button-on-edit-wheel-scree…
bc35260 
…n' of https://github.com/BinaryStudioAcademy/bsa-2024-bebalance into 467-feat-add-discard-changes-button-on-edit-wheel-screen
@iqbalalayubbi iqbalalayubbi added bug Something isn't working frontend labels Sep 27, 2024
@iqbalalayubbi iqbalalayubbi added this to the bebalance-release-6 milestone Sep 27, 2024
@iqbalalayubbi iqbalalayubbi self-assigned this Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@WilhelmWesser WilhelmWesser Awaiting requested review from WilhelmWesser WilhelmWesser is a code owner

@anogire anogire Awaiting requested review from anogire anogire is a code owner

@revenets revenets Awaiting requested review from revenets revenets is a code owner

@ViktoriiaKozopas ViktoriiaKozopas Awaiting requested review from ViktoriiaKozopas ViktoriiaKozopas is a code owner

@fshabanov fshabanov Awaiting requested review from fshabanov fshabanov is a code owner

Requested changes must be addressed to merge this pull request.

Assignees

@iqbalalayubbi iqbalalayubbi

Labels
bug Something isn't working frontend
Projects
None yet
Milestone
bebalance-release-6
Development

Successfully merging this pull request may close these issues.
2 participants
@iqbalalayubbi @fshabanov