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

When accept the terms in the account registration form, the phone field clears up #1173

Open
alexanmtz opened this issue Feb 3, 2025 · 8 comments · May be fixed by #1176
Open

When accept the terms in the account registration form, the phone field clears up #1173

alexanmtz opened this issue Feb 3, 2025 · 8 comments · May be fixed by #1176
Assignees
Labels
bug good first issue help wanted react Task that require react knowledge
Milestone

Comments

@alexanmtz
Copy link
Member

alexanmtz commented Feb 3, 2025

Bug description

When accept the terms in the account registration form,

How to reproduce the bug

  • Run the project locally (Follow README for instructions)
  • Create an account (Follow README - Account activation section for account activation locally)
  • Go to /#/profile/user-account/payouts
  • Type a phone number
  • Switch on accept terms
  • The Phone field clears

Expected results

  • The form should keep the actual phone data

Actual results

  • The form clears phone field and you have to type again

Screenshots/recordings

Image

Tips

@alexanmtz alexanmtz added bug good first issue help wanted react Task that require react knowledge labels Feb 3, 2025
@alexanmtz alexanmtz added this to the v2.0 milestone Feb 3, 2025
@alexanmtz alexanmtz added this to Gitpay Feb 3, 2025
@github-project-automation github-project-automation bot moved this to To do in Gitpay Feb 3, 2025
Copy link

dosubot bot commented Feb 3, 2025

Hey @alexanmtz! I'm here to help you with any bugs, questions, or contributions. Let's squash this bug together! Dosu

The issue you're experiencing might be related to the handleAgreeTerms function, which is triggered when the terms are accepted. This function updates the agreeTermsCheck state, but it should not affect the phone field directly [1].

To investigate further, you should check if there are any side effects or state updates in the component that might inadvertently reset the form fields when the agreeTermsCheck state changes. Ensure that the state management logic for the form fields, particularly the phone field, is isolated from the logic that handles the terms acceptance.

Additionally, verify if there are any lifecycle methods or hooks that might be causing the form fields to reset when the terms are toggled. If the issue persists, consider reviewing the component's state management to ensure that the phone field's state is not being reset unintentionally.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@alexanmtz
Copy link
Member Author

@LevisNgigi
Copy link

Hey @alexanmtz can i have a look into this? Would love to get assigned.

@parthikjadav
Copy link

Hey Alexandre Magno i can fix this issue. but how do i get my payment i dont have paypal id

@alexanmtz
Copy link
Member Author

@LevisNgigi hey Levis you're now assigned and you can start to work on it 👌

@alexanmtz
Copy link
Member Author

Hey @parthikjadav, I assigned @LevisNgigi to this one, but we will have more soon, and I will happily assign it to you.

You should activate your account and bank account to receive the payouts directly, and you don't need Paypal for these bounties paid by credit card.

You can activate your account here: https://gitpay.me/#/profile/user-account/payouts

@LevisNgigi
Copy link

@LevisNgigi hey Levis you're now assigned and you can start to work on it 👌

Thank you.Will be solving it.

@deekshatomer
Copy link

Hey @alexanmtz i can fix this issue please assign me..

@LevisNgigi LevisNgigi linked a pull request Feb 13, 2025 that will close this issue
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug good first issue help wanted react Task that require react knowledge
Projects
Status: To do
Development

Successfully merging a pull request may close this issue.

5 participants
@alexanmtz @LevisNgigi @parthikjadav @deekshatomer and others