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

Allow people to be responsible for bill actions #83

Open
fgregg opened this issue Mar 16, 2017 · 2 comments
Open

Allow people to be responsible for bill actions #83

fgregg opened this issue Mar 16, 2017 · 2 comments

Comments

@fgregg
Copy link
Contributor

fgregg commented Mar 16, 2017

Right now, all bill actions have an organization attribute

organization, organization_id
____The organization that this action took place within.

This seems not quite right for the actions of 'signing' or 'vetoing' done by the executive.

Here are the two ways, I've approached this. Neither seem quite right to me.

            bill_action = {'description' : 'Veto',
                           'date' : action_date,
                           'organization' : 'Office of the Mayor',
                           'classification' : 'executive-veto')

I don't really like this approach because the mayor is not the one vetoing legislature because he holds a position in the office of the mayor. He can do it because he is the mayor of the "City of Chicago".

So this is my current approach.

            bill_action = {'description' : 'Veto',
                           'date' : action_date,
                           'organization' : 'City of Chicago',
                           'classification' : 'executive-veto',
                           'related_entitites = [{'name' : 'Rahm Emanuel, 'entity_type': 'person'}                          

I like this better, but "City of Chicago" also doesn't quite seem like the right container.

I think I would like the following to be legal, note the absence of organization"

            bill_action = {'description' : 'Veto',
                           'date' : action_date,
                           'person' : 'Rahm Emanuel',
                           'classification' : 'executive-veto')

Thoughts?

@jpmckinney
Copy link
Member

I wasn't involved in authoring the Bill spec so paging @jamesturk

@jamesturk
Copy link
Member

sorry, missed this, I think this makes sense & could be implemented using RelatedEntityBase without much trouble

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

3 participants