BioDrop
BioDrop copied to clipboard
[FEATURE] Add location to in-person events
Description
Maybe we could add a location to the in person events?

Screenshots
No response
Additional information
No response
It's great having you contribute to this project
Welcome to the community :nerd_face:If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.
Great point. Later with the data, we can also add to the live map to see which conferences are happening nearby.
Can I try to do this one please @ChinmayMhatre and @Pradumnasaraf?
Hey ! Can I try to work on this issue ? @Pradumnasaraf
I think the data we need to capture is...
"location": {
"road": "",
"city": "",
"state": "",
"country": ""
}
Please add to the docs also
I think the data we need to capture is...
"location": { "road": "", "city": "", "state": "", "country": "" }
Please add to the docs also
okay @eddiejaoude I got some idea about it. Where can I discuss about it with you ?
Hey @eddiejaoude, can you kindly assign me?
Hello @eddiejaoude, I'll be so happy to work on this issue. Kindly let me work on this.
Can I try to do this one please @ChinmayMhatre and @Pradumnasaraf?
Hey @ShuvamChatterjee28 I can't assign it to you. But you can go ahead with this. If you want any help let us know. Thank you!
Hey, @ShuvamChatterjee28 you can go ahead and others, please look for other issues.
Thank you!
Hey @Pradumnasaraf! @ShuvamChatterjee28 is already assigned and working on an issue i.e. issue number #4223. You can have a look at it.
For #4223, the PR is created. Thanks. Working on this issue now.
Idk! what kind of system is this that you want to contribute to Open Source? Some folks are assigned more than one issue which is fine but if you are asking for an issue again and again but no one is assigning you. That's so cool! @Pradumnasaraf
Hey, @ShuvamChatterjee28,
@ShuvamChatterjee28 commented on the issue 1st, this is why they are been assigned. Also, I didn't know that they are already assigned to an issue, and people can work on 1-2 issues together.
You can look for other open issues. I know you are an enthusiast and want to contribute. Hope you understand.
Cool no problem @Pradumnasaraf and also I have found an issue. Where can I raise it ?
Cool no problem @Pradumnasaraf and also I have found an issue. Where can I raise it ?
Here - https://github.com/EddieHubCommunity/LinkFree/issues/new/choose
Hey @Pradumnasaraf and @eddiejaoude, I have made a working prototype of the location parameter in events. Also I have updated the docs too. Please have a look at the screenshot and let me know if you like it or not -->
(The Docs containing the
location
parameter. Now all the required flags are false, because I have set the code like, if isVirtual is set as true && isInPerson is false, then by fault the location will be remote)

Please suggest, if I should make any changes in the layout or not!
Hey @ShuvamChatterjee28 , I think it would be best if you created a PR with your solution. The changes could be discussed over on the PR. Thanks
Got it @ChinmayMhatre, will create a PR. Thanks.
Yes, you can raise a draft PR!
Hey @ShuvamChatterjee28 just had a doubt what if event is in hybrid mode??? The logic will still show remote.
Nope, it is updated now.
@eddiejaoude I would like to work on this
@ashutosh887 please read the history of the issue timeline, you will see it is already assigned and a pull request has already been created.
Please have a look at the issue list and select an issue that
- is not already assigned (unless more than 2 weeks with no activity and we can reassign it)
- has no existing pul request
- also the issue has the label
status: ready for dev
Thanks @eddiejaoude I'll keep this in mind
PR has been merged