8000 Avatars won’t display if the app is deployed under a sub-path · Issue #386 · wekan/wekan · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Avatars won’t display if the app is deployed under a sub-path #386

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
Closed
mquandalle opened this issue Nov 3, 2015 · 2 comments
Closed

Avatars won’t display if the app is deployed under a sub-path #386

mquandalle opened this issue Nov 3, 2015 · 2 comments
Labels
Bug Feature:User-accounts:Avatars Targets:Raw-hosting

Comments

@mquandalle
Copy link

Deploying Wekan under a sub-path of a domain name was recently implemented in devel (see #133) and will be shipped in the next version of Wekan.

I strongly suspect this “avatar not displaying” bug comes from Collection-fs not taking into account the ROOT_URL environment variable correctly to support prefixed URLs. I will probably open a PR to support it (which should be fairly similar to the support introduced in Flow-Router in kadirahq/flow-router#329)
@mquandalle mquandalle added this to the Release 0.10 milestone Dec 18, 2015
@mquandalle
Copy link
Author

I was slightly wrong on the problem description, I reported the bug to CollectionFs with a corrected explanation at Meteor-Community-Packages/Meteor-CollectionFS#884.

@mquandalle mquandalle removed this from the Release 0.10 milestone Dec 18, 2015
@mquandalle mquandalle changed the title Avatars won’t display if the app is deboyed under a sub-path Avatars won’t display if the app is deployed under a sub-path Dec 20, 2015
@eemeli eemeli mentioned this issue Feb 27, 2017
Closed
7 tasks
@xet7
Copy link
Member
xet7 commented May 7, 2017

Moved to #973

@xet7 xet7 closed this as completed May 7, 2017
@xet7 xet7 mentioned this issue May 7, 2017
Closed
@zypA13510 zypA13510 mentioned this issue Jul 16, 2018
Closed
xet7 added a commit that referenced this issue Aug 22, 2018
@xet7
- Remove suburl from beginning of avatar file path, so that avatar im…
7e0bc1e 
…ages

  don't get broken when root-url changes to different sub-url.
  This does not change avatar urls in database, instead this
  fixes url on the fly after loading avatar url from database.

Thanks to xet7 !

Closes #1776,
closes #386
xet7 added a commit that referenced this issue Aug 22, 2018
@xet7
- Remove avatar url fix javascript version, it breaks too easily.
dfd26e4 
  Same can be done with Caddy.

Thanks to xet7 !

Closes #1776,
closes #386
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug Feature:User-accounts:Avatars Targets:Raw-hosting
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@xet7 @mquandalle
0