transparentdemocracy / voting-data

Voting behavior data extracted from plenary reports of the Belgian federal government.
5 stars 1 forks source link

Issues found in alfa test of the website #50

Open sandervh14 opened 1 month ago

sandervh14 commented 1 month ago

I just tested our website locally. I think these are important changes we need to make to make the website experience better:

karel1980 commented 1 month ago

I've changed title parsing so newlines aren't lost

karel1980 commented 1 month ago

also updated frontend to preserve newlines in titles (white-space: pre-wrap)

karel1980 commented 1 month ago

@sandervh14 can you provide examples for bullets 2 & 3 so we know what to focus on exactly?

karel1980 commented 1 month ago

image Screenshot for newlines in titles (this is a newline I added myself for testing)

sandervh14 commented 1 month ago

Nice. 🔥

I updated my issue description above (https://github.com/transparentdemocracy/voting-data/issues/50#issue-2321798181).

sandervh14 commented 3 weeks ago

New ones:

[ ] quite a few recent plenaries with "Er waren geen stemmingen", is this correct? [ ] motion on 28/3/324 with number 479 (unsupported number of motions so far) has no document reference and therefore also no summary.

karel1980 commented 3 weeks ago

I've checked plenaries 308, 307, 306, 305, 303 and 301 which didn't have any votes according to the website. Of these, only 301 actually had votes. I'll add a test case for this.

karel1980 commented 3 weeks ago

Extraction for 301 fails because the plenary doesn't open with a level 1 header (h1 or css class) and the extraction algorithm expects this.