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

Fix react error on share dialog #21475

Closed
germain-gg opened this issue Mar 18, 2022 · 8 comments · Fixed by matrix-org/matrix-react-sdk#8170
Closed

Fix react error on share dialog #21475

germain-gg opened this issue Mar 18, 2022 · 8 comments · Fixed by matrix-org/matrix-react-sdk#8170
Labels
A-Share good first issue Good for newcomers O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users T-Defect

Comments

@germain-gg
Copy link
Contributor

Steps to reproduce

  1. Hover over an event
  2. Select "share" in the message action bar to open the share dialog

Outcome

What did you expect?

no error

What happened instead?

In the console I can see

Warning: You provided a `checked` prop to a form field without an `onChange` handler. This will render a read-only field. If the field should be mutable use `defaultChecked`. Otherwise, set either `onChange` or `readOnly`.

Operating system

No response

Browser information

No response

URL for webapp

No response

Application version

No response

Homeserver

No response

Will you send logs?

No

@germain-gg germain-gg added T-Defect S-Tolerable Low/no impact on users A-Share O-Uncommon Most users are unlikely to come across this or unexpected workflow good first issue Good for newcomers labels Mar 18, 2022
@yaya-usman
Copy link
Contributor

yaya-usman commented Mar 25, 2022

hey @gsouquet can i get assigned this issue?

@SimonBrandner
Copy link
Contributor

@yaya-usman, I've assigned you to the issue. If you need any help/have any questions be sure to visit #element-dev:matrix.org!

@yaya-usman
Copy link
Contributor

@SimonBrandner thanks, let me start working on it

@SimonBrandner
Copy link
Contributor

SimonBrandner commented Mar 27, 2022

2022-03-27_20-00

Let me know if it is clearer now

@yaya-usman
Copy link
Contributor

2022-03-27_20-00

Let me know if it is clearer now

Yes,thanks

@yaya-usman
Copy link
Contributor

@SimonBrandner I have fixed this issue since, but i am having issues pushing my changes to my forked version so i could make a pull request, it's giving me remote push rejected

@yaya-usman
Copy link
Contributor

@SimonBrandner I have fixed this issue since, but i am having issues pushing my changes to my forked version so i could make a pull request, it's giving me remote push rejected

The thing is i only cloned the main repo the element-web and followed the guide in setting up the local version by linking matrix-react-sdk and matrix-js-sdk repos, so do i need to fork the supporting repos as well? For instance i only made changes to matrix-react-sdk, so i want to push my changes from there since element-web is linked to it, but somehow it's not working.

@SimonBrandner
Copy link
Contributor

so do i need to fork the supporting repos as well?

Yes, you need to fork all the repos, if you make changes to the react-sdk, you only push the changes and make PR from there.

Though, I would suggest asking these questions in https://matrix.to/#/#element-dev:matrix.org

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Share good first issue Good for newcomers O-Uncommon Most users are unlikely to come across this or unexpected workflow S-Tolerable Low/no impact on users T-Defect
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants