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

#115: added handler for sync bill error #116

Merged
merged 1 commit into from
May 8, 2019
Merged

Conversation

lazarus1331
Copy link
Collaborator

No description provided.

@lazarus1331 lazarus1331 added the bug label May 7, 2019
@lazarus1331 lazarus1331 self-assigned this May 7, 2019
@ghost ghost added the in progress label May 7, 2019
Copy link
Collaborator

@annaderengowski annaderengowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will definitely catch the error. Do we have any other way to grab the correct legislator in the event of multiples?

@lazarus1331
Copy link
Collaborator Author

Apparently its including sponsors from other chambers, but the API doesn't provide enough details to match those names. It inconsistently uses full names and last names only, and doesn't include the chamber nor ID of the sponsor.

@lazarus1331 lazarus1331 merged commit ac0ee42 into master May 8, 2019
@ghost ghost removed the in progress label May 8, 2019
@lazarus1331 lazarus1331 deleted the lazarus1331/115 branch February 27, 2020 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants