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

Incorrect MUR sort order #5982

Open
1 task
patphongs opened this issue Sep 18, 2024 · 1 comment · May be fixed by #6008
Open
1 task

Incorrect MUR sort order #5982

patphongs opened this issue Sep 18, 2024 · 1 comment · May be fixed by #6008
Assignees
Milestone

Comments

@patphongs
Copy link
Member

patphongs commented Sep 18, 2024

What we’re after

The sort order for MURs seem to be incorrect. This seems to be the case when current MURs end and archived MURs start. Example website link: https://www.fec.gov/data/legal/search/murs/?search=&case_respondents=&case_min_open_date=&case_max_open_date=&case_min_close_date=&case_max_close_date=&offset=3080&limit=20&sort=#results-murs

Screenshot 2024-09-18 at 1 38 16 PM

Completion criteria

  • MURs and archived MURs are sorted correctly
@fec-jli
Copy link
Contributor

fec-jli commented Oct 4, 2024

Note:
1)The Current MURs are stored in case_index, the Arch MURs are stored in arch_mur_index
2)Because both MURs are sorted independently when display on the website, the sort order may sometimes be incorrect.
3)So far there are total 7533 MURs(Arch MURs=4448 and Current MURs=3085)
4)Some examples:
ex 1: case_no= 2804(Arch MUR) -- case_no=2804R(Current MUR)
ex 2: case_no= 3620(Arch MUR) -- case_no=3620(Current MUR)
Image

ex 3: some Current MURs with same case_serial , but different case_no (xxxxR)
7894R; 4313R
Image

5)will create new view: fecmur.arch_current_mur_sorting_vw to combine both Current MURs and Arch MURs
with integer column 'case_serial' for sorting.

@fec-jli fec-jli linked a pull request Oct 5, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 📥 Assigned
Development

Successfully merging a pull request may close this issue.

3 participants