Skip to content
This repository has been archived by the owner on Jun 14, 2022. It is now read-only.

Add request to Bridge Troll API #3

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Add request to Bridge Troll API #3

wants to merge 1 commit into from

Conversation

LauraKirby
Copy link
Member

@LauraKirby LauraKirby commented Jul 5, 2017

  • Remove reference to locally stored data and use Bridge Troll API
  • Update Readme to include instructions for getting set up locally
  • Remove console.log of events count since it is not currently included within JSON response. It looks like adding it within Bridge Troll will not be too difficult. I have already figured it out, if it is something we should move forward with.
  • To do: identify any additional items we would like to have from the JSON response and discuss how that would be represented on the map.

@LauraKirby LauraKirby changed the title Add json Add request to Bridge Troll API Jul 5, 2017
@LauraKirby LauraKirby force-pushed the add-json branch 2 times, most recently from 5dae90d to f9815bf Compare July 6, 2017 18:05
@LauraKirby LauraKirby requested a review from tigarcia July 6, 2017 18:16
@ultrasaurus
Copy link
Member

thanks for this PR! I don't think it'll work yet because some locations don't yet have geo data in Bridge Troll (maybe the "external" events?)

For example:

  {
    "url": "http:\/\/www.meetup.com\/Mad-Railers\/events\/119755612\/",
    "title": "Ruby on Rails Outreach Workshop for Women",
    "location": {
      "name": "Madison Ruby & MadRailers",
      "city": "Madison, WI USA"
    },
    "organizers": "Jim Remsik & Desi McAdam",
    "sessions": [
      {
        "starts_at": "2013-08-21"
      }
    ],
    "organization": "RailsBridge",
    "workshop": true
  },

@ultrasaurus
Copy link
Member

waiting on: railsbridge/bridge_troll#499

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

Successfully merging this pull request may close these issues.

2 participants