Skip to content

Move raven_placeholder app under backend/raven_placeholder/#6706

Merged
pashiav merged 5 commits into
issue-6571from
issue-6705
Jul 16, 2025
Merged

Move raven_placeholder app under backend/raven_placeholder/#6706
pashiav merged 5 commits into
issue-6571from
issue-6705

Conversation

@pashiav

@pashiav pashiav commented Jun 18, 2025

Copy link
Copy Markdown
Contributor

Fixes #6705

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

Summary of Changes:

  • Moved raven_placeholder to specifyweb/backend/raven_placeholder/
  • Updated SPECIFY_APPS in settings.py to include specifyweb.backend.raven_placeholder

This is the 5th 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 functionality with main.

  • Visit the Home Page (url: .../specify/)
  • Open Developer Tools (Right click the site -> Inspect) -> Network tab
  • Reload the page and look through the Name column
  • Confirm that raven.min.js is not present

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

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

  • Confirm that raven.min.js is not present

Works as expected!

@pashiav
pashiav requested a review from a team July 2, 2025 18:16

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

  • Confirm that raven.min.js is not present

Looks good

Base automatically changed from issue-6701 to issue-6571 July 16, 2025 20:52
@pashiav
pashiav merged commit 5fcbba0 into issue-6571 Jul 16, 2025
14 checks passed
@github-project-automation github-project-automation Bot moved this from 📋Back Log to ✅Done in General Tester Board Jul 16, 2025
@pashiav
pashiav deleted the issue-6705 branch July 16, 2025 20:52
@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 raven_placeholder app under backend/raven_placeholder/

4 participants