The current install documentation in the INSTALL.txt and on the project page is incorrect for Drupal 8. I found that some different Drush 8 commands are needed to set the origin and other module settings that used to be set with drush variable-set. The new syntax for Drush 8 and Drupal 8 is as follows:

drush config-set stage_file_proxy.settings origin http://example.com

Replace http://example.com with domain you want to use as the origin.

CommentFileSizeAuthor
#3 instructions_2762353.patch1.63 KBjalpesh

Comments

sk33lz created an issue. See original summary.

greggles’s picture

Thanks, 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?

jalpesh’s picture

Status: Active » Needs review
StatusFileSize
new1.63 KB

I 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.

Loque’s picture

$config['stage_file_proxy.settings']['origin'] = 'http://example.com'; // no trailing slash

Does 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 : )

chrowe’s picture

This 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.com

twfahey’s picture

Status: Needs review » Reviewed & tested by the community

This 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.

  • axel.rutz committed 32e99fa on 8.x-1.x authored by jalpesh
    Issue #2762353 by jalpesh: Drupal 8 Install Instructions
    
geek-merlin’s picture

Status: Reviewed & tested by the community » Fixed

Thanks everyone!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

rick_p’s picture

@sk33lz, Nice! Worked like a charm.

ipwa’s picture

@axel.rutz it would be great to add this to the project description.

greggles’s picture

Status: Closed (fixed) » Active

@ipwa - can you provide proposed html changes for the project page?

ipwa’s picture

@greggles proposing replacing the 'Simple install and configuration for Drupal 7' section with this: https://codepen.io/ipwa/pen/ePqdyz

ipwa’s picture

greggles’s picture

Status: Active » Fixed

Thanks 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.

ipwa’s picture

Thanks @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!

greggles’s picture

I see, ok. I think Drupal.org's text format processing was adding those classes. Glad to have it fixed :)

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.