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.

Sign up for GitHub

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

Jump to bottom

Make sure no SayIt Speaker is associated with a blacklisted PopIt IDs #69

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Make sure no SayIt Speaker is associated with a blacklisted PopIt IDs #69

wants to merge 1 commit into from

Commits on Nov 26, 2015

  1. Make sure no SayIt Speaker is associated with a blacklisted PopIt IDs 

    Even if a PopIt ID was blacklisted, a speech might still end up
being associated with that person because there would still be a
Speaker object with the exact name that's being resolved which
has its person attribute set to a Person with the blacklisted
PopIt ID.

This commit makes sure that any Speaker with a person attribute
with a blacklisted PopIt ID has that person attribute set to None
when the ImportJson object is initialized.

Fixes mysociety/pombola#1740
    @mhl
    mhl committed Nov 26, 2015
    1 Configuration menu
    Copy the full SHA
    fe4c8d1 View commit details
    Browse the repository at this point in the history