Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Media link #292

Closed
Madeorsk opened this issue Oct 25, 2018 · 1 comment
Closed

Media link #292

Madeorsk opened this issue Oct 25, 2018 · 1 comment
Labels
A: Backend Code running on the server C: Bug Something isn't working

Comments

@Madeorsk
Copy link
Member

I just started a new local instance for testing, and when I upload a media, I see a two "/static" in the URL.

Screen capture of a the link with double static.

So I looked in the database and the file_path saved for the current record contains a static/.

Screen capture of the database record with a field containing static

Plume version (and/or commit): Commit b28c1a7568dd9ba38adc616eecae7cd2d5c4f2fe
@Madeorsk Madeorsk added C: Bug Something isn't working A: Backend Code running on the server P: Medium labels Oct 25, 2018
@elegaanz
Copy link
Member

I think we shouldn't add /static/ here, since it is already in self.file_path: https://github.com/Plume-org/Plume/blob/master/plume-models/src/medias.rs#L70

trinity-1686a added a commit that referenced this issue Oct 27, 2018
@trinity-1686a
Fix incorrect path for uploaded medias
8eb5291 
fix #292
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A: Backend Code running on the server C: Bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Madeorsk @elegaanz