bcgov / bcgov/entity

legal api - remove NAICS endpoint related artifacts (do this last)

Open
#11,387 1 comment 0 reactions 0 assignees View on GitHub
ENTITY - DO NOT USE
Dominant language
JavaScript
Stars
23
Forks
62
Avg merge
24m
Merged PRs (30d)
1

Description

Note: This is a placeholder ticket to update necessary BE components when the NAICS service is available as a separate service. It currently resides in the legal api.

**This task should be done last. i.e. after separate NAICS service is up and Create/Edit UI and the legal api is using this new NAICS service**

**TODOs**

- [ ] remove two GET NAICS endpoints (resource and model classes, tests)
- [ ] remove `NAICS_YEAR` and `NAICS_VERSION` from config
- [ ] create alembic script to remove `naics_structures` and `naics_elements` table in `lear` db

Contributor guide

No contributing guide indexed for this repository

Research direction

First confirm that the separate NAICS service is available and that the Create/Edit UI and legal API use it. Then locate the two GET NAICS resource and model classes and their tests, the configuration entries for NAICS_YEAR and NAICS_VERSION, and the alembic migrations for the lear database. Done means those endpoints, artifacts, configuration values, and the naics_structures and naics_elements tables are removed with tests passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
flask, postgres
Domain
api, backend, database
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.