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

GranicusSite code based on IQM2 site ? #108

Open
jsfenfen opened this issue Feb 3, 2022 · 1 comment
Open

GranicusSite code based on IQM2 site ? #108

jsfenfen opened this issue Feb 3, 2022 · 1 comment

Comments

@jsfenfen
Copy link

jsfenfen commented Feb 3, 2022

I noticed the code for "Granicus" sites here seems targeted towards Granicus sites that use "IQM2" in the URL. This is inconsistent, I think, with the naming conventions in #52 (about Granicus sites) and #53 (about IQM2 sites); specifically #52 seems to consider "Granicus" sites to be those without the IQM2 in the url.

See this commit of the Granicus code; it notes that this code was run and tested on Brookhaven GA, an IQM2 site.

I can confirm this code does not work on a non IQM2 site; the first thing that breaks is trying to parse an entry title by split()-ing on a hyphen; the IQM2 sites have three args whereas the other Granicus sites have 2. ( In fact the Granicus sites sometimes do have 3 pieces when split by '-' but this only occurs when the meeting has been canceled. FWIW the RSS view of a "normal" granicus site looks like this.

@jsfenfen
Copy link
Author

jsfenfen commented Feb 3, 2022

You can see this in the sites that are tested as Granicus here: https://github.com/biglocalnews/civic-scraper/blob/master/tests/granicus_site.py

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

1 participant