Skip to content

Move export app under backend/export#6770

Merged
pashiav merged 6 commits into
issue-6571from
issue-6769
Aug 4, 2025
Merged

Move export app under backend/export#6770
pashiav merged 6 commits into
issue-6571from
issue-6769

Conversation

@pashiav

@pashiav pashiav commented Jun 25, 2025

Copy link
Copy Markdown
Contributor

Fixes #6769

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

Summary of Changes:

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

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

Notes:

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

DwCA Data Exporting in Specify 7 Documentation

Please cross check the following functionalities with main.

  • General testing of Exports (additional gen testing in addition to the ones below)
    Verify that you can:

    • Go to Queries
    • Click the Edit pencil icon on any saved Query
    • Click Export
    • See a JSON file has been downloaded

    Use a database with DwCA configured (e.g. KU_Fish):

    • Go to User Tools
    • Under Export, click Create DwC Archive
    • Make sure you select the DwCA and metadata from the same collection that you're in
    • Wait to receive a notification "DwCA export completed." then click Download
    • DwCA zip file is downloaded and the content inside of it is the same as on main
8mb.video-Qam-upLvTLun.mp4

@pashiav pashiav added this to the 7.12.0 milestone Jun 25, 2025
@pashiav pashiav self-assigned this Jun 25, 2025
@github-project-automation github-project-automation Bot moved this to 📋Back Log in General Tester Board Jun 25, 2025
@pashiav pashiav linked an issue Jun 25, 2025 that may be closed by this pull request
@pashiav
pashiav marked this pull request as ready for review June 25, 2025 16:31
@pashiav
pashiav requested review from a team and CarolineDenis June 25, 2025 16:32
@pashiav pashiav changed the title Move export under backend/export Move export app under backend/export Jun 25, 2025

@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 testing of Exports (additional gen testing in addition to the ones below)
    • See a JSON file has been downloaded
    • DwCA zip file is downloaded and the content inside of it is the same as on main

Looks good and everything is working but there is one error message connected to the following bug report that pops up - #2920

@bhumikaguptaa
bhumikaguptaa requested a review from a team July 2, 2025 19:44

@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 testing of Exports (additional gen testing in addition to the ones below)
    • See a JSON file has been downloaded
    • DwCA zip file is downloaded and the content inside of it is the same as on main

Looks good!

@emenslin
emenslin requested a review from a team July 3, 2025 14:43
Base automatically changed from issue-6763 to issue-6571 July 16, 2025 20:56
@pashiav

pashiav commented Jul 16, 2025

Copy link
Copy Markdown
Contributor Author

@bhumikaguptaa Appreciate you pointing that out! It looks like it’s unrelated to this PR - would you mind giving this another quick review?

@pashiav
pashiav requested a review from bhumikaguptaa July 16, 2025 21:00

@lexiclevenger lexiclevenger 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 testing of Exports (additional gen testing in addition to the ones below)
    • See a JSON file has been downloaded
    • DwCA zip file is downloaded and the content inside of it is the same as on main

Looks good! I didn't run into issues with either type of export.

@pashiav
pashiav merged commit 4a2df55 into issue-6571 Aug 4, 2025
14 checks passed
@pashiav
pashiav deleted the issue-6769 branch August 4, 2025 17:55
@github-project-automation github-project-automation Bot moved this from 📋Back Log to ✅Done in General Tester Board Aug 4, 2025
@CarolineDenis CarolineDenis modified the milestones: 7.12.0, 7.11.2 Aug 5, 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 export app under backend/export/

5 participants