-
-
Notifications
You must be signed in to change notification settings - Fork 323
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
Conversation
…/shubhamkakkar/src/segment.js, issue #58 resolved
@shubhamkakkar Please create another PR specifically for #58 as seen in our Working on a PR guide. |
hey, I am sorry for the inconveniences caused, I will soon raise a new PR
for issue #58 as well. This particular PR is having some conflicts I
suppose and doesn't follow the PR pattern of *galio*
for most of the other issues mentioned to be resolved by this PR, i have
raised custom PR for each of them so that you can check each working
individually!!
Issue #58 's PR will be raised in a day or two ( PS: actually my exams are
going on 😅 )
…On Tue, Aug 20, 2019, 4:57 PM Petru Alin Gheorghe ***@***.***> wrote:
@shubhamkakkar <https://github.com/shubhamkakkar> Please create another
PR specifically for #58 <#58> as
seen in our Working on a PR guide
<https://github.com/galio-org/galio/wiki/Working-on-a-PR>.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#105?email_source=notifications&email_token=ABGTFMOG75JAJZEI3PI6UCDQFPIK7A5CNFSM4IDUIS5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4V6QBA#issuecomment-522971140>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABGTFMMALHNF2XR3UFOXKTLQFPIK7ANCNFSM4IDUIS5A>
.
|
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: thanks! ❤️ |
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:
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. |
@palingheorghe #125 is a new PR dealing only with Segment !! its a work in progress |
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