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

Disambiguate fladistctapp docket numbers #1136

Closed
grossir opened this issue Aug 28, 2024 · 2 comments
Closed

Disambiguate fladistctapp docket numbers #1136

grossir opened this issue Aug 28, 2024 · 2 comments

Comments

@grossir
Copy link
Contributor

grossir commented Aug 28, 2024

From freelawproject/courtlistener#4256

We return docket numbers as we find them in the HTML. However, we should prepend f"{district_number}D" to the docket number, as is done in the opinion files themselves.

image

These 2 are asigned to the same docket:
image
image
@grossir
Copy link
Contributor Author

grossir commented Aug 29, 2024

Some more evidence of the full docket numbers
image
image

image
image

grossir added a commit to grossir/juriscraper that referenced this issue Aug 29, 2024
@grossir
fix(fladistctapp): prepend district code to dockets
883a198 
Solves freelawproject#1136

- Docket numbers returned in the HTML do not have a district component, which does exist inside the PDFs. This causes different cases to have the same docket number, being incorrectly merged
- Removed `fladistctapp` file
- Refactored scrapers
- Now collecting disposition and per_curiam
- Updated test files
@grossir grossir mentioned this issue Aug 29, 2024
Merged
grossir added a commit to grossir/juriscraper that referenced this issue Aug 29, 2024
@grossir
fix(fladistctapp): prepend district code to dockets
a3c0f24 
Solves freelawproject#1136

- Docket numbers returned in the HTML do not have a district component, which does exist inside the PDFs. This causes different cases to have the same docket number, being incorrectly merged
- Removed `fladistctapp` file
- Refactored scrapers
- Now collecting disposition and per_curiam
- Updated test files
@grossir
Copy link
Contributor Author

grossir commented Sep 3, 2024

This is now working
https://www.courtlistener.com/opinion/10103065/robert-anthony-zaccaro-v-state-of-florida/?q=court_id%3Afladistctapp&type=o&order_by=dateFiled%20desc&stat_Published=on

@grossir grossir closed this as completed Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@grossir