codeforamerica / brigade-information

Data about Code for America brigades and other civic tech organizations for the CfA API
MIT License
17 stars 63 forks source link

Switch Hack for LA to topic-based projects list #106

Closed themightychris closed 4 years ago

themightychris commented 4 years ago

Hack for LA wants to drive their projects list from a topic tag

Does it make sense to merge changes like this into organizations.json, or will that break existing things? Would it make more sense to just fork off the index's orgs list instead of continuing to merge it from here?

tdooner commented 4 years ago

Is it possible to special-case Hack for LA in the new project scraper rather than merging this? At least until we determine if this approach works.

themightychris commented 4 years ago

No need to do that, the crawler projects organizations.json into it's own data branch as-is first: https://github.com/codeforamerica/brigade-project-index/tree/cfapi/orgs/v1

We can leave that going, but branch off and make this change there, and switch the crawler to use that branch

When organizations.json changes here at all, we'll see a new commit pop up and can merge it (manually for now)

themightychris commented 4 years ago

@tdooner if the result of merging this though would be that Hack for LA gets skipped by CfAPI, that might be what they prefer in the short term

tdooner commented 4 years ago

Closing this in light of your other PRs.