Skip to content

(bug) fix local Vite asset prefix#529

Merged
angelina-ji merged 1 commit into
masterfrom
bug/local-dev-vite-prefix
May 3, 2026
Merged

(bug) fix local Vite asset prefix#529
angelina-ji merged 1 commit into
masterfrom
bug/local-dev-vite-prefix

Conversation

@Ammaar-Alam
Copy link
Copy Markdown
Member

@Ammaar-Alam Ammaar-Alam commented May 1, 2026

What does this PR do?

  • Aligns Django Vite dev URLs with the configured Vite base.
  • Fixes the local blank or unstyled preview caused by the missing dist static URL prefix.
  • Keeps the change scoped to local dev asset serving.

Why?

The local preview could render blank because Django generated dev-server asset URLs that did not match Vite's configured path prefix.

How to test

  • Start the Django server with DJANGO_VITE_DEV_MODE=True.
  • Start Vite locally.
  • Verify the app loads frontend assets from /static/dist/....

Screenshots

Dashboard Before

Dashboard before

Dashboard After

Dashboard after

Verification

  • Verified Vite serves http://localhost:3000/static/dist/src/landing.jsx.
  • Screenshot verification used localhost with DATABASE_URL=postgres:///tigerpath.

Note

This entire PR was automatically implemented, tested, and written by Codex.

@Ammaar-Alam Ammaar-Alam changed the title Fix local Vite asset prefix (bug) fix local Vite asset prefix May 1, 2026
@Ammaar-Alam
Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. More of your lovely PRs please.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Copy link
Copy Markdown
Contributor

@angelina-ji angelina-ji left a comment

Choose a reason for hiding this comment

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

lgtm

@angelina-ji angelina-ji merged commit e0a0795 into master May 3, 2026
1 check passed
@angelina-ji angelina-ji deleted the bug/local-dev-vite-prefix branch May 3, 2026 18:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants