-
Notifications
You must be signed in to change notification settings - Fork 15
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
Comments
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
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.
The text was updated successfully, but these errors were encountered: