Skip to content

Move report_runner app under backend/report_runner#6765

Merged
pashiav merged 6 commits into
issue-6571from
issue-6763
Jul 16, 2025
Merged

Move report_runner app under backend/report_runner#6765
pashiav merged 6 commits into
issue-6571from
issue-6763

Conversation

@pashiav

@pashiav pashiav commented Jun 25, 2025

Copy link
Copy Markdown
Contributor

Fixes #6763

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

Summary of Changes:

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

This is the 11th 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

Please cross check the following functionalities with main.

  • Create a record that generates a report/invoice on save (e.g. Loan or Gift)
  • Verify the report opens in a new tab

  • Click Reports in the navigation bar
  • Run a valid report
  • Verify the report opens in a new tab

@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 15:21
@pashiav
pashiav requested review from a team and CarolineDenis June 25, 2025 15:21
@pashiav pashiav changed the title Move report_runner under backend/report_runner Move report_runner app under backend/report_runner Jun 25, 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.

Testing instructions

  • Verify the report opens in a new tab
  • Verify the report opens in a new tab

Looks good!

@emenslin
emenslin requested a review from a team July 2, 2025 18:26

@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.

  • Verify the report opens in a new tab
  • Verify the report opens in a new tab

Works as expected!

@bhumikaguptaa
bhumikaguptaa requested a review from a team July 2, 2025 20:03
Base automatically changed from issue-6756 to issue-6571 July 16, 2025 20:54
@pashiav
pashiav merged commit cbe23c5 into issue-6571 Jul 16, 2025
14 checks passed
@pashiav
pashiav deleted the issue-6763 branch July 16, 2025 20:56
@github-project-automation github-project-automation Bot moved this from 📋Back Log to ✅Done in General Tester Board Jul 16, 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 report_runner app under backend/report_runner/

4 participants