relic_bazaar icon indicating copy to clipboard operation
relic_bazaar copied to clipboard

Invite Friends

Open himanshusharma89 opened this issue 4 years ago • 15 comments

Use dynamic links to share with your friends to invite. Follow up pf #50

himanshusharma89 avatar Mar 06 '21 08:03 himanshusharma89

Since this is the follow up of #50 Do I have to work on this as well?

nb9960 avatar Mar 08 '21 04:03 nb9960

Yes @nb9960, but one person can be assigned to one issue at a time.

himanshusharma89 avatar Mar 08 '21 04:03 himanshusharma89

Yes @nb9960, but one person can be assigned to one issue at a time.

Okay, but featuring dynamic links using firebase requires setting up firebase first which in itself is a separate issue. Or do one thing generate URL using firebase and provide me that link so that I can start working on #50

nb9960 avatar Mar 08 '21 05:03 nb9960

@nb9960 I'll add you to the firebase console and integration is almost done, just need to review the PR. Share me your Gmail id.

himanshusharma89 avatar Mar 08 '21 09:03 himanshusharma89

@nb9960 I'll add you to the firebase console and integration is almost done, just need to review the PR. Share me your Gmail id.

[email protected]

nb9960 avatar Mar 08 '21 10:03 nb9960

Hey @himanshusharma89 here do we only have to share the dynamic links that are generated in issue #50 ? Like by providing a share button on the products details page?

anotherwebguy avatar Mar 10 '21 17:03 anotherwebguy

Hey @himanshusharma89 i can work on this issue if issue #50 is merged, also can u fill me with details like here we sharing the links to the products right? So i need to pass the products info in the link ? Also i need to work on that link generation plus sharing part? Also after opening app from link it should take you to the product detail page right? Plz assign me this issue i would like to work on it. Here's my mail u can add me on firebase console [email protected]

Hey @anotherwebguy PR for dynamic link integration is not merged yet. For resolving this issue you just have to create a invite button (maybe in drawer) and pass URL as relicbazaar.page.link/ (without any parameter as this isn't about sharing product, I have already done that part). Other than that as I have already integrated dynamic link you don't have to make any changes through the firebase console or in the codebase to handle the link on app launch

nb9960 avatar Mar 10 '21 17:03 nb9960

Hey @himanshusharma89 i can work on this issue if issue #50 is merged, also can u fill me with details like here we sharing the links to the products right? So i need to pass the products info in the link ? Also i need to work on that link generation plus sharing part? Also after opening app from link it should take you to the product detail page right? Plz assign me this issue i would like to work on it. Here's my mail u can add me on firebase console [email protected]

Hey @anotherwebguy PR for dynamic link integration is not merged yet. For resolving this issue you just have to create a invite button (maybe in drawer) and pass URL as relicbazaar.page.link/ (without any parameter as this isn't about sharing product, I have already done that part). Other than that as I have already integrated dynamic link you don't have to make any changes through the firebase console or in the codebase to handle the link on app launch

Yeah @nb9960 i saw your pr i got my doubt cleared so i can work on this part right @himanshusharma89 ?

anotherwebguy avatar Mar 10 '21 17:03 anotherwebguy

@anotherwebguy As I mentioned earlier one person can work on a single issue at a time.

himanshusharma89 avatar Mar 10 '21 18:03 himanshusharma89

@anotherwebguy As I mentioned earlier one person can work on a single issue at a time.

Hey @himanshusharma89 but at the moment i am not assigned any issue in relic_bazar project.plz assign me this issue I means my work in issue #39 was only to make the screen responsive which i already did and u merged that pr So at present i am not assigned any issue

anotherwebguy avatar Mar 10 '21 23:03 anotherwebguy

@anotherwebguy The issue is still not resolved but I'll assign new issues to you. By responsive it means we don't have to add a custom height for the widget instead use the Flex.

himanshusharma89 avatar Mar 11 '21 04:03 himanshusharma89

@anotherwebguy The issue is still not resolved but I'll assign new issues to you. By responsive it means we don't have to add a custom height for the widget instead use the Flex.

Okay sir i'll do that

anotherwebguy avatar Mar 11 '21 05:03 anotherwebguy

@anotherwebguy The issue is still not resolved but I'll assign new issues to you. By responsive it means we don't have to add a custom height for the widget instead use the Flex.

So now can u assign me this issue as i've now made an pr for issue #39

anotherwebguy avatar Mar 11 '21 06:03 anotherwebguy

So now can u assign me this issue as i've now made an pr for issue #39

This issue will be taken care of later on as it is dependant upon the Dynamic Links. And we need to create the UI first.

himanshusharma89 avatar Mar 11 '21 15:03 himanshusharma89

@himanshusharma89 please assign this issue to me

subhangi2731 avatar May 24 '21 07:05 subhangi2731