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

issue #104, issue #101, issue #106 #97 and #58 resolved #105

Closed
wants to merge 4 commits into from
Closed

issue #104, issue #101, issue #106 #97 and #58 resolved #105

wants to merge 4 commits into from

Conversation

shubhamkakkar
Copy link
Contributor

The new codes for the respective components could be found under the folder src of folder shubhamkakkar

To see the working of the components on Expo Application, refer https://github.com/shubhamkakkar/galio/tree/examples

@shubhamkakkar shubhamkakkar changed the title Components are converted to fucntional Componetns, issue #104 resolved Components are converted to fucntional Componetns, issue #104, issue #101 resolved Jul 15, 2019
@shubhamkakkar shubhamkakkar changed the title Components are converted to fucntional Componetns, issue #104, issue #101 resolved issue #104, issue #101, issue #106 and #97 resolved Jul 17, 2019
…/shubhamkakkar/src/segment.js, issue #58 resolved
@shubhamkakkar shubhamkakkar changed the title issue #104, issue #101, issue #106 and #97 resolved issue #104, issue #101, issue #106 #97 and #58 resolved Aug 4, 2019
@palingheorghe
Copy link
Collaborator

@shubhamkakkar Please create another PR specifically for #58 as seen in our Working on a PR guide.

@shubhamkakkar
Copy link
Contributor Author

shubhamkakkar commented Aug 20, 2019 via email

@palingheorghe
Copy link
Collaborator

palingheorghe commented Aug 28, 2019

Hi @shubhamkakkar ! When are we having the PR for issue #58 ? You pushed everything else besides that one. Please push the PR with the name: Components/Segment so I'd know which one is it. You're doing a great job!

thanks! ❤️

@palingheorghe
Copy link
Collaborator

Hi @shubhamkakkar ! We still didn't got all the PRs.. The PRs for issue #101 , #106, #97 and #58 are still missing.

As a priority I'd prefer if you'd follow the next priority list:

  1. Components: Segment #58 is the most important one for v0.6, you've pushed it here and then sent me a Snack link but I can't verify it so I'd actually prefer a PR. (I'm going to keep trying with that Snack link but honestly, a work-in-progress PR is a lot better)
  2. Switch trackColor not working #101
  3. remove left icon #97
  4. Improper Alignment of navbar buttons (right) #106

Honestly no. 2, 3, 4 are a lot less important than the no.1 item on the priority list.

Thank for your help ❤️🔥 and please let's focus on creating more organized PRs because this has gotten quite out of hand for me, it's starting to be really disorganized.

tl;dr: #58 is the most important PR right now.

@shubhamkakkar
Copy link
Contributor Author

@palingheorghe #125 is a new PR dealing only with Segment !! its a work in progress

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants