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

Any hacktoberfest enthusiastic can contribute PRs on this repo #32

Closed
SURYAKANTSHARMA opened this issue Oct 1, 2019 · 8 comments
Closed

Comments

@SURYAKANTSHARMA
Copy link
Owner

SURYAKANTSHARMA commented Oct 1, 2019

Rules

  • Can reduce the launch time of pickerView (Please attach statistics if work)
  • Can add voice over accessibility
  • Test cases are recommended.
  • Add documentation of public apis by jazz (https://github.com/realm/jazzy)
  • Add value to project
  • No Spam
@megandsouza03
Copy link

Hey Suryakant

What are some of the areas/issues I can work on?

@SURYAKANTSHARMA
Copy link
Owner Author

Hey Megan, Thanks for interest.
You can work on

  1. add test coverage
  2. Add documentation by jazz (https://github.com/realm/jazzy)
  3. Any think you see is missing 👍

@megandsouza03
Copy link

Perfect, I'll add some tests soon!

Is there any feature enhancement you would like me to work on?

@SURYAKANTSHARMA
Copy link
Owner Author

  • Can reduce the launch time of pickerView (Please attach statistics if work)
  • Can add voice over accessibility

@thisIsTheFoxe
Copy link

Hey, just to let you know: I’m currently working on Accessibility support and will PR probably some time tmr (in ~12-16h)

Like the project and that you think about stuff like Accessibility! ^^

@SURYAKANTSHARMA
Copy link
Owner Author

Yes please @thisIsTheFoxe Add and generate PR. 💯

@megandsouza03
Copy link

Hey Suryakant

I'll be writing UITests for the repo, would that be fine?
Or you have something else in mind?

@SURYAKANTSHARMA
Copy link
Owner Author

Yes, sure go for it.

SURYAKANTSHARMA added a commit that referenced this issue Oct 23, 2019
added accessibility labels + placeholder grammar fix #32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants