Skip to content

[FIX] Attachments and avatars not rendered if deployed on subfolder#22290

Merged
tassoevan merged 3 commits into
developfrom
fix/subfolder_attachments
Jun 11, 2021
Merged

[FIX] Attachments and avatars not rendered if deployed on subfolder#22290
tassoevan merged 3 commits into
developfrom
fix/subfolder_attachments

Conversation

@gabriellsh

@gabriellsh gabriellsh commented Jun 8, 2021

Copy link
Copy Markdown
Member

Proposed changes (including videos or screenshots)

Issue(s)

#18759
#19542

Steps to test or reproduce

  • Run develpment server with this command ROOT_URL=http://localhost:3000/<SUBFOLDER_NAME> meteor
  • Go to Setting > General and change the Site URL setting to match the new address.
  • Avatars and attachments should be working
    PS.: Old quotes will have the avatars broken, since the data from quotes is simply copied instead of referenced. This will only affect servers that CHANGED URL to run on a subfolder

Further comments

@gabriellsh gabriellsh requested a review from a team June 8, 2021 20:09
@tassoevan tassoevan merged commit 274511c into develop Jun 11, 2021
@tassoevan tassoevan deleted the fix/subfolder_attachments branch June 11, 2021 18:09
@JoshMcCullough

Copy link
Copy Markdown
Contributor

Which release is this expected to go in?

@dougfabris

Copy link
Copy Markdown
Member

Which release is this expected to go in?

There is a great probability to be present in the next one, 3.16

@sampaiodiego sampaiodiego changed the title [FIX] Attachments not rendered if deployed on subfolder [FIX] Attachments and avatars not rendered if deployed on subfolder Jun 15, 2021
@sampaiodiego sampaiodiego added this to the 3.15.1 milestone Jun 16, 2021
sampaiodiego pushed a commit that referenced this pull request Jun 21, 2021
Co-authored-by: Tasso Evangelista <tasso.evangelista@rocket.chat>
This was referenced Jun 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants