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

get_members() returns meetup join date instead group join date #20

Open
limegimlet opened this issue Feb 1, 2018 · 13 comments
Open

get_members() returns meetup join date instead group join date #20

limegimlet opened this issue Feb 1, 2018 · 13 comments
Assignees

Comments

@limegimlet
Copy link

find_groups('R-ladies Paris')
returns
id name urlname created members status organizer lat lon city
1 2.04e⁷ R-Ladi… rladies-… 2016-09-19 17:39:51 298 active Gabriela… 48.9 2.34 Paris

however
get_members('rladies-paris')
returns members with join dates prior to group create date!
id name bio status joined city country state lat lon
1 8.37e⁶ Sarah… I miss skii… active 2008-11-15 22:13:36 Paris fr <NA> 48.9 2.34
2 1.45e⁷ Gabri… <NA> active 2011-04-20 00:57:39 San … us CA 37.8 -122

@ledell ledell self-assigned this Feb 1, 2018
@ledell
Copy link
Member

ledell commented Feb 1, 2018

@limegimlet Thanks for the report! I checked to make sure joined actually means "when the user joined the group" rather than "when the user joined meetup.com", and according to the docs, it sounds like it's supposed to be when they joined the group.

It seems to be a bug. What I see is that R-Ladies Paris was created on "2016-09-19 08:39:51 PDT" and I have verified that this date is correct, so then I checked the dates that people joined and I see this:

> mem <- get_members('rladies-paris')
> range(mem$joined)
[1] "2007-03-13 08:14:36 PDT" "2018-01-30 02:22:53 PST"

We do some parsing of the date (maybe it's being parsed incorrectly), and if that's not the issue then either the meetup.com API is returning the wrong values or their documentation is wrong (and it's actually showing the date they joined meetup.com).

You're the first entry in the members table, and it says you joined on "2008-11-15"... is that the date you joined meetup.com, by chance?

@limegimlet
Copy link
Author

limegimlet commented Feb 7, 2018 via email

@missaugustina missaugustina self-assigned this Apr 11, 2018
@missaugustina
Copy link
Member

I started looking at reproducing this awhile ago, I'm digging that back up.

@missaugustina
Copy link
Member

I have reproduced and demonstrated this issue. The value you want is "group_profile.created".

http://rpubs.com/rladiespdx/meetup-member-dates

I also confirmed, the docs are weird.

Under Sort "joined" refers to sorting the member list by when the member joined the group:
Sort
joined
Time member joined this group

Under Response:

joined
Time member joined, represented as milliseconds since the epoch

Under Group Profile (I think this is what you want):

created
The time this member joined the Group, represented as milliseconds since the epoch

@missaugustina
Copy link
Member

I need to look at the code to figure out what makes the most sense as far as a patch proposal.

@missaugustina
Copy link
Member

Removed help wanted since I'm working on this :)

@missaugustina
Copy link
Member

working on issue #31 so i can have test coverage for this one.

@RickPack
Copy link
Collaborator

@missaugustina, are you still working on this? Might it help if I had a look? I am not skilled with formal testing but I have used this package a great deal (please see http://bit.ly/2NEHVGn). So I could at least explore capturing group_profile.created.

@ledell
Copy link
Member

ledell commented May 16, 2019

@missaugustina @RickPack Anyone have an update on this? Someone ran into this bug again today and so it reminded me to come here and check the status. Let me know and we can re-assign the issue if necessary. Thanks!

@orchid00
Copy link

I came to the same problem today,
is there any updates?

@RickPack
Copy link
Collaborator

I missed my name mention. This issue was not assigned to me. It looks like @missaugustina was going to address the issue.

@orchid00
Copy link

@RickPack @missaugustina @ledell
I solved it doing this:
1 download json file of members per chapter
2. convert it to a csv
3. mutate the column group_profile.created
mutate(date_joined = lubridate::floor_date(as.POSIXct( group_profile.created/1000, tz = "UTC", origin = "1970-01-01")))

I guess the the mutate on step 3 should be added to the function get_members, and that would be it.

@maelle
Copy link
Contributor

maelle commented Mar 22, 2021

@ledell what needs to happen here? Do we need to not use what the API returns for that date?

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

No branches or pull requests

6 participants