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

ZA: some committee appearances from November 2014 are missing #1597

Closed
mhl opened this issue Jan 21, 2015 · 1 comment · Fixed by #1611
Closed

ZA: some committee appearances from November 2014 are missing #1597

mhl opened this issue Jan 21, 2015 · 1 comment · Fixed by #1611

Comments

@mhl
Copy link
Contributor

mhl commented Jan 21, 2015

As reported by Zaheedah, these appearances are missing:

PC Basic Education

Ms D van der Walt (DA) –11 and 19 Nov
Ms N Gina (Chair, ANC)– 11 Nov
Ms A Lovemore (DA) – 11 and 19 Nov
Ms J Basson (ANC) – 11 Nov

*Committee appearances have not been updated since the 4 Nov meeting.

PC Health

Ms J Dunjwa (Chair, ANC) – 12 and 19 Nov
Dr W James (DA) – 12 and 26 Nov
Mr A Shaik Emam – 19 and 26 Nov

*Committee appearances have not been updated since the 5 Nov meeting.

PC Police

Mr F Beukman (Chair, ANC) – 12 and 19 Nov
Mr J Maake (ANC) – 12 and 19 Nov
Ms D Kohler-Barnard – 19 Nov

*Committee Appearances not updated since 5 November meeting.

@mhl mhl added the 3 - Now label Jan 21, 2015
mhl added a commit that referenced this issue Feb 2, 2015
…kers

We have changed the form of IDs in the South African PopIt instance
several times, but without removing the entries corresponding to old
PopIt IDs from the popit-django popit_person table. This meant that the
name resolution on importing new committee proceedings could associate
the speeches with various SayIt speakers corresponding to different
entries in that table.

This commit introduces a script to hopefully clean up that mess; it
tries to remove any row with an deprecated format ID from the
popit_person table, safely removing any SayIt Speaker associated with
that row.

This should fix #1597 and hopefully #1601.

The underlying issue (that Pombola ZA stores the identity of people in
five different places) is being addressed by #1594.
mhl added a commit that referenced this issue Feb 2, 2015
…kers

We have changed the form of IDs in the South African PopIt instance
several times, but without removing the entries corresponding to old
PopIt IDs from the popit-django popit_person table. This meant that the
name resolution on importing new committee proceedings could associate
the speeches with various SayIt speakers corresponding to different
entries in that table.

This commit introduces a script to hopefully clean up that mess; it
tries to remove any row with an deprecated format ID from the
popit_person table, safely removing any SayIt Speaker or EntityName
associated with that row. It also removes any rows from this table that
can't be associated with any Pombola core Person any more, so long as
they are not referenced any more.

This should fix #1597 and hopefully #1601.

The underlying issue (that Pombola ZA stores the identity of people in
five different places) is being addressed by #1594.
mhl added a commit that referenced this issue Feb 2, 2015
…kers

We have changed the form of IDs in the South African PopIt instance
several times, but without removing the entries corresponding to old
PopIt IDs from the popit-django popit_person table. This meant that the
name resolution on importing new committee proceedings could associate
the speeches with various SayIt speakers corresponding to different
entries in that table.

This commit introduces a script to hopefully clean up that mess; it
tries to remove any row with an deprecated format ID from the
popit_person table, safely removing any SayIt Speaker or EntityName
associated with that row. It also removes any rows from this table that
can't be associated with any Pombola core Person any more, so long as
they are not referenced any more.

This should fix #1597 and hopefully #1601.

The underlying issue (that Pombola ZA stores the identity of people in
five different places) is being addressed by #1594.
@mhl mhl closed this as completed in #1611 Feb 2, 2015
@mhl
Copy link
Contributor Author

mhl commented Feb 2, 2015

I've checked and all of these appearances are on the site now after that fix.

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

Successfully merging a pull request may close this issue.

1 participant