Skip to content

Move workbench app under backend/workbench/#6834

Merged
pashiav merged 8 commits into
issue-6571from
issue-6833
Aug 6, 2025
Merged

Move workbench app under backend/workbench/#6834
pashiav merged 8 commits into
issue-6571from
issue-6833

Conversation

@pashiav

@pashiav pashiav commented Jun 30, 2025

Copy link
Copy Markdown
Contributor

Fixes #6833

This PR reorganizes the project structure by moving the workbench app into a new backend/workbench/ directory under specifyweb/.

Summary of Changes:

  • Moved workbench to specifyweb/backend/workbench/
  • Updated all import paths to reflect the new location
  • Updated SPECIFY_APPS in settings.py to include specifyweb.backend.workbench

This is the 16th PR in a broader backend reorganization. No frontend behavior or functionality was changed.

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 pr to documentation list
  • Add automated tests
  • Add a reverse migration if a migration is present in the PR

Testing instructions

View WorkBench documentation here & specifically WorkBench Grid Locality Tools documentation here.

Please cross check the following functionalities with main.

  • General WorkBench testing (additional gen testing in addition to the ones below)
    Verify that you:
    New Data Set:

    • Click on WorkBench in the navigation bar
    • Click on Create New -> Select any base table (e.g. Collection Object)
    • Add columns and valid column mapping (e.g. catalogNumber) then click Save
    • Add a new record/cell -> Click Save -> Validate
    • Upload the data set
    • Data set uploaded with the correct fields & is identical to main functionality

    Importing a Data Set:

    • Click on WorkBench in the navigation bar
    • Import a File to Create a New Data Set
    • Resulting data set is the same as on main
    • Click Data Mapper
    • Map corresponding fields and Save
    • Click on Tools
    • Click on Upload Plan -> Export
    • JSON file downloads and contains the expected information (same as main)
    • Close the Upload Plan dialog then click Export
    • TSV file downloads and contains the expected information (same as main)

    WorkBench Grid Locality Tools:

    • General testing of Converting Coordinates
    • General testing of GeoLocate
    • General testing of GeoMap

@pashiav pashiav added this to the 7.12.0 milestone Jul 2, 2025
@pashiav pashiav self-assigned this Jul 2, 2025
@pashiav pashiav changed the title Move workbench app under backend/workbench Move workbench app under backend/workbench/ Jul 2, 2025
@pashiav
pashiav marked this pull request as ready for review July 2, 2025 18:20
@pashiav
pashiav requested review from a team and CarolineDenis July 2, 2025 18:20

@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 WorkBench testing (additional gen testing in addition to the ones below)
    • Data set uploaded with the correct fields & is identical to main functionality
    • Resulting data set is the same as on main
    • JSON file downloads and contains the expected information (same as main)
    • TSV file downloads and contains the expected information (same as main)
    • General testing of Converting Coordinates
    • General testing of GeoLocate
    • General testing of GeoMap

Works as expected!

@bhumikaguptaa
bhumikaguptaa requested a review from a team July 2, 2025 21:05
Vegito2367

This comment was marked as spam.

@github-project-automation github-project-automation Bot moved this from 📋Back Log to Dev Attention Needed in General Tester Board Jul 3, 2025

@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 WorkBench testing (additional gen testing in addition to the ones below)
    • Data set uploaded with the correct fields & is identical to main functionality
    • Resulting data set is the same as on main
    • JSON file downloads and contains the expected information (same as main)
    • TSV file downloads and contains the expected information (same as main)
    • General testing of Converting Coordinates
    • General testing of GeoLocate
    • General testing of GeoMap

Looks good, all behaviors seem to match main.

@pashiav pashiav modified the milestones: 7.12.0, 7.11.2 Aug 4, 2025
Base automatically changed from issue-6822 to issue-6571 August 6, 2025 13:52
@pashiav
pashiav merged commit 596af67 into issue-6571 Aug 6, 2025
21 checks passed
@pashiav
pashiav deleted the issue-6833 branch August 6, 2025 14:03
@github-project-automation github-project-automation Bot moved this from Dev Attention Needed to ✅Done in General Tester Board Aug 6, 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 workbench app under backend/workbench/

5 participants