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.

Sign up for GitHub

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

Jump to bottom

Meetup API: removing RSS2JSON dependency #87

Open
MoribundMedium opened this issue Sep 6, 2021 · 0 comments
Open

Meetup API: removing RSS2JSON dependency #87

MoribundMedium opened this issue Sep 6, 2021 · 0 comments
Assignees
@Hellrungj
Labels
enhancement New feature or request

Comments

@MoribundMedium
Copy link
Collaborator

MoribundMedium commented Sep 6, 2021 •
edited
Loading

Acceptance Criteria

We are currently accessing Meetup API data using the rss2json.com website (line 28 in pingEvents.js). This is convenient for converting the XML from the Meetup API to a JSON format, but it means that we have a dependency on RSS2JSON's services, and if they ever go down or change their terms of use we could be in trouble. If we go directly to the Meetup API and convert it from XML to JSON in our own service, we won't need to rely on RSS2JSON.

The Node package xml2js may be helpful for converting: https://www.npmjs.com/package/xml2js. It will require some additional massaging of the resulting JSON to match what we were originally getting from RSS2JSON.

Related Features
@Hellrungj Hellrungj added the enhancement New feature or request label Sep 30, 2021
@Hellrungj Hellrungj self-assigned this Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Hellrungj Hellrungj

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Hellrungj @MoribundMedium