Support for static url

[marianoeramirez]

I want to merge this change because. I think is important to have the ability to edit the location of the assets with the STATIC_URL environment parameter like the dashboard application.

Pull Request Checklist

1. All visible strings are translated with proper context.
2. All data-formatting is locale-aware (dates, numbers, and so on).
3. The changes are tested.
4. The code is documented (docstrings, project documentation).
5. Changes are mentioned in the changelog.

[netlify]

Deploy preview for saleor-storefront-stage processing.

Building with commit f2c3031

https://app.netlify.com/sites/saleor-storefront-stage/deploys/5ec5786c171f230007389ec1

[orzechdev]

Hi, thank you for contributing! It is definitely important. May I ask you for a rebase with master to resolve conflicts? We will then see that all tests pass and we will be able to merge changes.

[lgtm-com]

This pull request introduces 2 alerts when merging 071c2ea into b296500 - view on LGTM.com

new alerts:

• 1 for Duplicate property
• 1 for Overwritten property

[marianoeramirez]

@orzechdev I merge and resolved the conflicts, The only test that is not passing is the build because it requires the API_URL. But not sure how to solve this. Do you have any suggestion?

[orzechdev]

@Bomba1990 Oh, great! We do not have configured CircleCI to run tests with forked repos, but I have run it locally and it passes all of them. Unfortunately, I see there is unnecessarily duplicated devtool: "source-map", and also output property, which should be removed in the config/webpack/config.base.js - as reported by lgtm bot. May you fix it too?

[marianoeramirez]

@orzechdev Done, I Fixed the reported by LGTM.

[orzechdev]

Tests pass locally:

[marianoeramirez]

@dominik-zeglen I reviewed the requested, let me now if left any other comment.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[marianoeramirez]

@dominik-zeglen @orzechdev Do we have any update on this?