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

Remove unnecessary tables and field from db schema #1623

Merged
merged 5 commits into from
Jul 5, 2024

Conversation

spwoodcock
Copy link
Member

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation
  • πŸ§‘β€πŸ’» Refactor
  • βœ… Test
  • πŸ€– Build or CI
  • ❓ Other (please specify)

Describe this PR

  • We had many extra tables and fields we don't use.
  • Originally I wasn't sure if we may need them in the future.
  • I am more certain that we won't now - deleted.

Checklist before requesting a review

[optional] What gif best describes this PR or how it makes you feel?

@spwoodcock spwoodcock added backend Related to backend code devops Related to deployment or configuration labels Jul 3, 2024
@spwoodcock spwoodcock requested a review from Sujanadh July 3, 2024 18:30
@spwoodcock spwoodcock self-assigned this Jul 3, 2024
@github-actions github-actions bot added migration Contains a DB migration and removed devops Related to deployment or configuration labels Jul 3, 2024
@spwoodcock spwoodcock force-pushed the build/simplify-schema branch from 60c22f6 to 2edc114 Compare July 4, 2024 16:35
@spwoodcock spwoodcock merged commit b14e199 into development Jul 5, 2024
5 checks passed
@spwoodcock spwoodcock deleted the build/simplify-schema branch July 5, 2024 08:07
spwoodcock added a commit that referenced this pull request Jul 8, 2024
* fix(hotfix): download of basemaps in ui, max zoom level 22 used for tms

* ci: update all gh-workflows to latest v1.6.0

* build: upgrade rclone --> v1 pin to avoid CVE-2024-24790

* build: remove unnecessary tables and field from db schema (#1623)

* build: remove unnecessary tables via migration & simplify schema

* refactor: remove references to removed tables in sqlalchemy models

* build: add IF EXISTS to DROP COLUMN in migration

* build: remove default columns from previous migration

* fix(backend): minor fixes to HTTPException on endpoints

* fix(backend): addded created date on the project response, set expiry of access token to 1 hour (#1633)

* docs: add placeholder for axiom repo activity

* fix(createProjectSlice): set generateProject & drawToggle status to default

* feat(commonUtils): isStatusSuccess function add

* test: get the detailed task history for a project (#1626)

* build(backend): add async-lru dep, remove cpuinfo dep

* fix: replace lru_cache with async for getting odk creds

* fix(splitTasks): dependency add to useEffect

* fix(createProjectService): halt project creation if api failure

---------

Co-authored-by: spwoodcock <[email protected]>
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sujan Adhikari <[email protected]>
Co-authored-by: Azhar Ismagulova <[email protected]>
spwoodcock added a commit that referenced this pull request Jul 8, 2024
* fix(hotfix): download of basemaps in ui, max zoom level 22 used for tms

* ci: update all gh-workflows to latest v1.6.0

* build: upgrade rclone --> v1 pin to avoid CVE-2024-24790

* build: remove unnecessary tables and field from db schema (#1623)

* build: remove unnecessary tables via migration & simplify schema

* refactor: remove references to removed tables in sqlalchemy models

* build: add IF EXISTS to DROP COLUMN in migration

* build: remove default columns from previous migration

* fix(backend): minor fixes to HTTPException on endpoints

* fix(backend): addded created date on the project response, set expiry of access token to 1 hour (#1633)

* docs: add placeholder for axiom repo activity

* fix(createProjectSlice): set generateProject & drawToggle status to default

* feat(commonUtils): isStatusSuccess function add

* test: get the detailed task history for a project (#1626)

* build(backend): add async-lru dep, remove cpuinfo dep

* fix: replace lru_cache with async for getting odk creds

* fix(splitTasks): dependency add to useEffect

* fix(createProjectService): halt project creation if api failure

---------

Co-authored-by: spwoodcock <[email protected]>
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sujan Adhikari <[email protected]>
Co-authored-by: Azhar Ismagulova <[email protected]>
spwoodcock added a commit that referenced this pull request Jul 8, 2024
* fix(frontend): prevent project generation if api failure

* Prevent project generation if api failure pt2 (#1635)

* fix(hotfix): download of basemaps in ui, max zoom level 22 used for tms

* ci: update all gh-workflows to latest v1.6.0

* build: upgrade rclone --> v1 pin to avoid CVE-2024-24790

* build: remove unnecessary tables and field from db schema (#1623)

* build: remove unnecessary tables via migration & simplify schema

* refactor: remove references to removed tables in sqlalchemy models

* build: add IF EXISTS to DROP COLUMN in migration

* build: remove default columns from previous migration

* fix(backend): minor fixes to HTTPException on endpoints

* fix(backend): addded created date on the project response, set expiry of access token to 1 hour (#1633)

* docs: add placeholder for axiom repo activity

* fix(createProjectSlice): set generateProject & drawToggle status to default

* feat(commonUtils): isStatusSuccess function add

* test: get the detailed task history for a project (#1626)

* build(backend): add async-lru dep, remove cpuinfo dep

* fix: replace lru_cache with async for getting odk creds

* fix(splitTasks): dependency add to useEffect

* fix(createProjectService): halt project creation if api failure

---------

Co-authored-by: spwoodcock <[email protected]>
Co-authored-by: Sam <[email protected]>
Co-authored-by: Sujan Adhikari <[email protected]>
Co-authored-by: Azhar Ismagulova <[email protected]>

* build: update to latest ms playwright image v1.45.1

---------

Co-authored-by: Nishit Suwal <[email protected]>
Co-authored-by: Sujan Adhikari <[email protected]>
Co-authored-by: Azhar Ismagulova <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Related to backend code migration Contains a DB migration
Projects
Development

Successfully merging this pull request may close these issues.

1 participant