Closed (fixed)
Project:
Stage File Proxy
Version:
8.x-1.x-dev
Component:
Documentation
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Jul 2016 at 15:55 UTC
Updated:
15 Nov 2018 at 14:59 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
gregglesThanks, sk33lz! I don't use the module on d8 so can't confirm. I pinged someone who I know does use it - hoping they'll provide confirmation.
BTW, can you provide a patch for the INSTALL.txt?
Comment #3
jalpesh commentedI think user can also set variable by following manually copy paste in settings.php file, which was already mention module INSTALL.txt file.
$config['stage_file_proxy.settings']['origin'] = 'http://example.com'; // no trailing slash
But, It is good to add Drush command in INSTALL.txt.
Comment #4
Loque commented$config['stage_file_proxy.settings']['origin'] = 'http://example.com'; // no trailing slashDoes not work for me for some reason (if I browse to configuration > system > stage file proxy settings, the origin box is empty.)
It seems to be very difficult to determine what the local setting array value would be for any of the admin settings (so they can be added to local settings). If anyone has any guidance on how to work out what the array values are I would be happy to document it somewhere (and also be incredibly grateful for the time saved - so time consuming and error prone to get developers of a site to log in and manually add values that could simply be stored in a file).
EDIT: The setting does actually work for me, however the Admin UI displays a different configuration value - not sure how/why yet : )
Comment #5
chrowe commentedThis works for me. Maybe it could be added to the project page along with the D7 drush instructions?
drush config-set stage_file_proxy.settings origin http://example.comComment #6
twfahey commentedThis works for me! I think it should for sure go into the docs / project page immediately. I know with D7 this was a great pointer to read on the project page, and with my D8 site, at first it was a bummer to not be able to quickly reference the drush command, but I just tested it and confirmed working! Great! Moving to Reviewed & tested by community.
Comment #8
geek-merlinThanks everyone!
Comment #10
rick_p commented@sk33lz, Nice! Worked like a charm.
Comment #11
ipwa commented@axel.rutz it would be great to add this to the project description.
Comment #12
greggles@ipwa - can you provide proposed html changes for the project page?
Comment #13
ipwa commented@greggles proposing replacing the 'Simple install and configuration for Drupal 7' section with this: https://codepen.io/ipwa/pen/ePqdyz
Comment #14
ipwa commentedComment #15
gregglesThanks so much, @ipwa!
I made those changes to the project page now - can you confirm it looks good? I had to do some manual editing to remove some unrelated classes/html tags that are not necessary, so I'm a little nervous I missed something in there, but it mostly appears good.
Comment #16
ipwa commentedThanks @greggles it looks great! I added those classes just to follow the patterns from the existing html, but it does make sense to clean it up. Yay!
Comment #17
gregglesI see, ok. I think Drupal.org's text format processing was adding those classes. Glad to have it fixed :)