AtB-AS / webshop

AtB Webshop / AtB Nettbutikk
https://nettbutikk.atb.no
European Union Public License 1.2
0 stars 1 forks source link

Extract assets into org and common folders #399

Closed toretefre closed 3 years ago

toretefre commented 3 years ago

Most changed files are images and illustrations that are extracted out of the static folder into orgs/assets folders outside the code tree, ready to be copied in using yarn setup <environment> <organization>.

Extracts assets such as images, logos and illustrations into static/common for common assets between organizations and static/org for organization specific assets.

This PR adds a requirement to run yarn setup <environment> <organization> before deploying, in the same way as done in the mobile app. Example yarn setup dev atb or yarn setup prodstaging nfk This command is also included in yarn build from now on, so things can be deployed without additional config steps.

Finally, the URL for privacy declarations and zone maps is configured through the orgs/<organization>.json file

github-actions[bot] commented 3 years ago

Visit the preview URL for this PR (updated for commit b6d2b15):

https://atb-webshop-staging--pr399-tefre-extract-org-as-lnrv8elj.web.app

(expires Sun, 24 Oct 2021 13:52:40 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

toretefre commented 3 years ago

Tror denne er klar for review nå. I all hovedsak gjør den organiseringen av assets for flere aktører mer strukturert og oversiktlig, samt at det innføres et ekstra skript som kjøres ved bygging. Foreløpig ikke så mye automatikk utover det, siden Nordland tenker å branche ut av main for å hardkode all teksten sin i påvente av språkstøtte i nettbutikken.

mikaelbr commented 3 years ago

Tror URL til personvernerklæring også burde være konfigurerbar. Og sone-kart.

mikaelbr commented 3 years ago

@toretefre Om denne er klar til review ville jeg ha tagga @gorandalum f.eks til review og endret tittel til å ikke være WIP

toretefre commented 3 years ago

Skal få inn URLene, så kan den merkes klar, godt forslag @mikaelbr 👍

toretefre commented 3 years ago

Vi har fortsatt tanker om å få dette inn i designsystemet, dette blir å regne som et steg på veien.