-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Comments
hey @gsouquet can i get assigned this issue? |
@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! |
@SimonBrandner thanks, let me start working on it |
@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. |
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 |
Steps to reproduce
Outcome
What did you expect?
no error
What happened instead?
In the console I can see
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
The text was updated successfully, but these errors were encountered: