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

DOT Lookups #602

Merged
merged 17 commits into from
Oct 11, 2023
Merged

DOT Lookups #602

merged 17 commits into from
Oct 11, 2023

Conversation

dpgraham4401
Copy link
Member

@dpgraham4401 dpgraham4401 commented Oct 11, 2023

Description

Adds DOT lookup capabilities to our application. includes the below components

  1. A new DotLookups model via the django ORM
  2. new query-able endpoints for
    • certain DOT lookups are quite voluminous such as the ID numbers. Our endpoints all the ID numbers to be search and by default (currently the only option) return the first 100 rows (or less)
  3. makes use of the the new Dot ID numbers endpoint in our react application which can be used when filling out the waste line form. This has a nice performance boost as before we just had the 2000+ options in a static file that was bundled and the form showed all 2000+ options at the same time which cased lagging (that was bugging me)

Issue ticket number and link

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@github-actions github-actions bot added client Related to front end workings (React/Redux) database changes that require modification to the database schema dependencies project dependency problems or modifications server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included labels Oct 11, 2023
@dpgraham4401 dpgraham4401 merged commit 34e36ce into USEPA:main Oct 11, 2023
9 checks passed
@dpgraham4401 dpgraham4401 deleted the dot_codes branch October 11, 2023 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Related to front end workings (React/Redux) database changes that require modification to the database schema dependencies project dependency problems or modifications server Related to backend workings (Django/Rest API) test Unit tests are, or need to be, included
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant