Skip to content

Move accounts app under backend/accounts/#6577

Merged
pashiav merged 151 commits into
mainfrom
issue-6571
Aug 12, 2025
Merged

Move accounts app under backend/accounts/#6577
pashiav merged 151 commits into
mainfrom
issue-6571

Conversation

@pashiav

@pashiav pashiav commented Jun 4, 2025

Copy link
Copy Markdown
Contributor

Fixes #6571

This PR combines multiple backend reorganization changes into one. It moves several backend apps under specifyweb/backend/ and updates imports, settings, and related code accordingly.

Summary of Changes

  • Moved multiple backend apps (e.g., accounts, permissions, and others) into specifyweb/backend/
  • Updated all import paths to reflect the new structure
  • Updated SPECIFY_APPS in settings.py to include new backend app locations
  • Adjusted Celery task autodiscovery logic for updated paths
  • Applied similar reorganization changes across other backend apps merged into this PR
  • No frontend behavior or functionality was changed

PRs Merged into This One

  1. Move permissions app under backend/permissions/ #6618
  2. Move attachment_gw app under backend/attachment_gw/ #6626
  3. Move barvis app under backend/barvis/ #6704
  4. Move raven_placeholder app under backend/raven_placeholder/ #6706
  5. Move notifications app under backend/notifications/ #6709
  6. Move express_search app under backend/express_search/ #6729
  7. Move hibernateboolsbackend app to backend folder #6735
  8. Move stats app under backend/stats/ #6743
  9. Move interactions app under backend/interactions/ #6758
  10. Move report_runner app under backend/report_runner #6765
  11. Move export app under backend/export #6770
  12. Move businessrules app under backend/businessrules #6778
  13. Move patches app under backend/patches #6794
  14. Move stored_queries app under backend/stored_queries #6826
  15. Move workbench app under backend/workbench/ #6834
  16. Move context app under backend/context #7109

Checklist

  • Self-review the PR after opening it to make sure the changes look good and
    self-explanatory (or properly documented)
  • Add relevant issue to release milestone
  • Add relevant documentation (Tester - Dev)
  • Add automated tests
  • Add a reverse migration if a migration is present in the PR

Testing instructions

  • General Accounts testing
  • General Permissions testing
  • General Attachments testing
  • General WorkBench testing
  • General Home Page/Taxon Tiles preferences testing
  • General Notifications testing
  • General Simple Search testing
  • General Boolean Field testing (yes/no or true/false fields, see Move hibernateboolsbackend app to backend folder #6735 for more information)
  • General Statistics testing
  • General Interactions testing
  • General Reports testing
  • General Exports testing
  • General Business Rules/Uniqueness Rules testing
  • General Migrations/Data Fixes verification (see Move patches app under backend/patches #6794 for more information)
  • General App Resources testing
  • General Queries testing

@pashiav pashiav added this to the 7.12.0 milestone Jun 4, 2025
@pashiav
pashiav requested review from a team and CarolineDenis June 4, 2025 19:43
@pashiav pashiav self-assigned this Jun 4, 2025
@github-project-automation github-project-automation Bot moved this to 📋Back Log in General Tester Board Jun 4, 2025
@pashiav pashiav linked an issue Jun 4, 2025 that may be closed by this pull request
@pashiav
pashiav marked this pull request as draft June 4, 2025 19:45
@pashiav
pashiav marked this pull request as ready for review June 10, 2025 14:42
@CarolineDenis

Copy link
Copy Markdown
Contributor

NOTES:
If testing locally, do a docker rebuilt

@combs-a combs-a left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • General Accounts Testing
    • Log in using a valid username and password
    • Open a collection and land on the Specify homepage
    • Log out successfully
    • Change your password while logged in and log back in using the new password

Didn't run into any issues, everything works as normal.

@combs-a
combs-a requested a review from a team June 24, 2025 16:51

@emenslin emenslin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing instructions

  • General Accounts Testing
    • Log in using a valid username and password
    • Open a collection and land on the Specify homepage
    • Log out successfully
    • Change your password while logged in and log back in using the new password

Looks good!

@emenslin
emenslin requested a review from a team June 30, 2025 16:58

@bhumikaguptaa bhumikaguptaa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • General Accounts Testing
    • Log in using a valid username and password
    • Open a collection and land on the Specify homepage
    • Log out successfully
    • Change your password while logged in and log back in using the new password

Works as expected.

pashiav added 9 commits July 16, 2025 15:52
Move `raven_placeholder` app under `backend/raven_placeholder/`
Move `notifications` app under `backend/notifications/`
Move `express_search` app under `backend/express_search/`
Move `hibernateboolsbackend` app to backend folder
Move `stats` app under `backend/stats/`
Move `interactions` app under `backend/interactions/`
Move `report_runner` app under `backend/report_runner`
Move `export` app under `backend/export`
Move `businessrules` app under `backend/businessrules`
@pashiav pashiav modified the milestones: 7.12.0, 7.11.2 Aug 4, 2025
pashiav and others added 6 commits August 6, 2025 08:51
Move `patches` app under `backend/patches`
Move `stored_queries` app under `backend/stored_queries`
Move `workbench` app under `backend/workbench/`
Move `context` app under `backend/context`

@alesan99 alesan99 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • General Accounts testing
  • General Permissions testing
  • General Attachments testing
  • General WorkBench testing
  • General Home Page/Taxon Tiles preferences testing
  • General Notifications testing
  • General Simple Search testing
  • General Boolean Field testing (yes/no or true/false fields)
  • General Statistics testing
  • General Interactions testing
  • General Reports testing
  • General Exports testing
  • General Business Rules/Uniqueness Rules testing
  • General Migrations/Data Fixes verification (see #6794 for more information)
  • General App Resources testing
  • General Queries testing

Everything I tried works 👍👍👍
Tried on KUBirds_2024_06_06.

@bhumikaguptaa bhumikaguptaa left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • General Accounts testing
  • General Permissions testing
  • General Attachments testing
  • General WorkBench testing
  • General Home Page/Taxon Tiles preferences testing
  • General Notifications testing
  • General Simple Search testing
  • General Boolean Field testing (yes/no or true/false fields, see #6735 for more information)
  • General Statistics testing
  • General Interactions testing
  • General Reports testing
  • General Exports testing
  • General Business Rules/Uniqueness Rules testing
  • General Migrations/Data Fixes verification (see #6794 for more information)
  • General App Resources testing
  • General Queries testing

I am unable to work on the "General Migrations/Data Fixes verification" because I don't have access to the local host. Apart from that, Everything looks good!

@emenslin emenslin left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • General Accounts testing
  • General Permissions testing
  • General Attachments testing
  • General WorkBench testing
  • General Home Page/Taxon Tiles preferences testing
  • General Notifications testing
  • General Simple Search testing
  • General Boolean Field testing (yes/no or true/false fields, see #6735 for more information)
  • General Statistics testing
  • General Interactions testing
  • General Reports testing
  • General Exports testing
  • General Business Rules/Uniqueness Rules testing
  • General Migrations/Data Fixes verification (see #6794 for more information)
  • General App Resources testing
  • General Queries testing

Looks good, however I cannot test migrations as I don't have a local instance set up on my laptop

@pashiav
pashiav merged commit dadc872 into main Aug 12, 2025
14 checks passed
@pashiav
pashiav deleted the issue-6571 branch August 12, 2025 15:31
@github-project-automation github-project-automation Bot moved this from 📋Back Log to ✅Done in General Tester Board Aug 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅Done

Development

Successfully merging this pull request may close these issues.

Reorganize accounts app under backend/accounts/

6 participants