-
-
Notifications
You must be signed in to change notification settings - Fork 740
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
Add more Collaborators to ActionSheetPicker-3.0 #348
Comments
Would be interested to help. |
Let's give it a try. |
@skywinder Sure. |
awesome! Thanks for a quick response.
I will try to fix them all, and feel free to participate. |
It would be important for me to understand what is the strategy for maintaining library. For example - how many iOS versions back support is planned? (in my opinion - supporting current and 2 versions back is reasonable target), is rewrite to Swift a goal or is good compatibility enough? (issues asking for Swift support? rewrite?). Better Swift interop can be addressed with better header comments (or with Swift wrapper insetad of full rewrite). Also I think many of the issues are asking to address very specific edge cases (that in my opinion should not be in the scope of the library, like #355). |
Cc @BubiDevs (since you are actively working with this project 👍) |
Would be interested to help. |
@skywinder , @Others |
Hi @skywinder! I'm interested to help on this project, since I'm currently use the repo in a project. As @vincedev noted, the repo needs some cleanup because there are some stuff that could be closed or removed. Can we discuss an initial clean-up plan and an evolution process for this repo? |
@BubiDevs Grazie! I really appreciate your help. 🙏 p.s. |
@vincedev please, feel free to attend to our init call. let's arrange it together. I will get an intro, and will help to the cleanup repo and show how it works! Thank you guys, you rock! |
Ok many thanks! I will add you on Telegram! 🚀 PS: I'm using this library in Syncrogest app, an app that I develop for a customer. |
Perfect. Just added your app to |
Hello! I updated pod, merged as much as I can. Now looks that it works on iOS 13 and all kind of iPads. Thank you, everyone, for the support! |
@xjki, @BubiDevs thank you for continuous support and resolving the issues. Much appreciated! Thanks for constant contributions and helping with answering to the questions. You help me a lot. 🖖So I will add you to collaborators. It will grant you access to repo management. Congrats, and much appreciated! So welcome to Collaborators family! 🎉 @vincedev it's clean and up now. So feel free to use it. The up-to-date version in |
Update about your questions:
|
Oh many thanks! Will be a pleasure to help on this repo! 🚀 |
@skywinder this is my plan to discuss the evolution process of the repo:
What do you think? PS: can we create a Slack workspace to discuss this general stuff? |
sounds good. Let's give a try! a 5 cents from me:
|
I'm not so used to
Ok, we can use Discord, no problem for me. I will create the Discord project and the dedicated issue soon :) |
Here is the link for Discord: https://discord.gg/jnYVY2 |
Invite doesn’t work feel free to to make a PR with permanent link, btw
On Fri, 13 Dec 2019 at 01:05, Andrea ***@***.***> wrote:
Here is the link for Discord: https://discord.gg/jnYVY2
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#348?email_source=notifications&email_token=AAZTOOXKFVYKBB7I2OGQQK3QYJVMLA5CNFSM4DMVEYS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGXK27Q#issuecomment-565095806>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAZTOOVXOPMZEJEPAR4HTALQYJVMLANCNFSM4DMVEYSQ>
.
--
---
Regards,
Petr Korolev
|
My fault, I used a link with an expiration day. I've made a PR with the right Discord link :) |
@skywinder I would be glad to help |
@abdelmagied94 thank you, glad to hear. I would be very helpful with the 2 following things:
|
Please, have a look on #460 |
@BubiDevs Please, have a look on #460 |
@skywinder I would be glad to help 🎉 |
@noorulain17 hooray! Are you using the library now? I would be glad. Can you please review open pull requests so that we can update the library for ios 14.0. |
@skywinder yeah I'm using this library in two of my apps that are live on AppStore. Sure, I'll review the other open PRs, though I've opened the iOS 14.0 one. |
would be glad to help you |
We had a crash. I'll also contribute if the ticket and solution do not exist yet. Thanks for this! |
Thank you, glad to hear~~ |
I want other people to feel like they own ActionSheetPicker-3.0, too, so it helps in the long run.
What I'm looking for is people who can:
All when you have time and feel like it. It's not meant to be a burden.
Thanks to @searls for the push to do this by this talk.
If you're interested in helping out – when it's convenient for you – comment on this issue and I'll add you as a committer with push access.
Cheers! 😄 💻 🌍
/cc @ainopara @delackner @velga @kashifhisam @TimCinel
The text was updated successfully, but these errors were encountered: