-
Notifications
You must be signed in to change notification settings - Fork 6
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
Combining efforts #15
Comments
Thanks @zvictor for pointing it out I'll be happy to add him as official maintainer on the repo if @sakerhetspolisen is okay with the same also if he doesn't mind I can create PR in my repo cherry picking changes from here (of course attributing @sakerhetspolisen as author) |
@zvictor thanks for reaching out and @harshzalavadiya thanks for your good work on |
@sakerhetspolisen awesome, I've added write access for https://github.com/hc-oss/react-web-share repository, feel free to create PR, we can release v3.x when your PR merges |
Hi again @harshzalavadiya I'm so sorry for not replying earlier. Are you still interested and willing to send me another invitation? I've completely forgot to accept the invitation and it seems like the old one has expired. |
Great work and beautiful aesthetic improvements, @sakerhetspolisen !
I am just wondering if you have considered creating a PR and merging your work with @harshzalavadiya's.
On one hand your project looks has a much better look, but his is better maintained. Can't we combine the efforts here? Everybody could benefit a lot! 😊
@harshzalavadiya what are your thoughts on a possible merge?
The text was updated successfully, but these errors were encountered: