When customizing the PWA, always keep in mind that you may want to upgrade from time to time and will have to merge incoming changes into your codebase.
We, therefore, suggest not to edit existing files to a large extent and to keep them as intact as possible.
In this document we provide a couple of rules to follow.
Generally speaking:
Keep modifications on existing files as minimal as possible!
It can be tempting to always modify existing templates, component and style files inline when doing customization.
However, when merging incoming upgrades the number of merge conflicts can possibly be large.
So if you want to upgrade to new PWA versions later, stick to the recommendations in this guide.
If modifications to standard files seem absolutely inevitable, it is good practice to mark the customizations with some // CUSTOMIZATION
or <!-- CUSTOMIZATION -->
comments.
Either mark a changed line like this or use it as a beginning and end marker for added blocks.
When initially setting up an Intershop PWA-based project it is not advisable to clone the complete GitHub repository of the Intershop PWA.
All that is needed initially is the master
branch that includes the released versions of the PWA.
This can be achieved with the following Git command.
git clone --single-branch --branch master --origin intershop https://github.com/intershop/intershop-pwa.git project-pwa
This command clones only the master
branch from the GitHub repository of the Intershop PWA with the remote name intershop
(not origin
that will be needed later on for the project's own remote Git repository) into a folder project-pwa
(instead of intershop-pwa
).
Based on this initial version of the Intershop PWA (the latest release), any project customizations can be started.
The PWA uses themes which include theme-specific
To start customizing, set a default theme for your customization with the script
node schematics/customization/add --default <theme-prefix>
It will:
angular.json
which is used to configure your individual theme. This default theme will be used instead of the existing B2B and B2C themes. It is possible to configure and run multiple themes next to each other, see Guide - Themes.package.json
and replace all active themes (if --default
is used)environment.<theme-prefix>.ts
which should be used for further theme configurationsrc/styles/themes/<theme-prefix>
schematics/src/helpers/override/schema.json
ish
to custom
.eslintrc.json
to support your new theme files (specifically, the project-structure
rule)Note
If only one theme is active, PM2 will run the theme-specific SSR process in cluster mode on the default port (see Building Multiple Themes).
After that we recommend to additionally use the prefix custom
in every component to further help identifying customized components.
$ ng g c shared/components/basket/custom-basket-display
CREATE src/app/shared/components/basket/custom-basket-display/custom-basket-display.component.ts (275 bytes)
...
Note
The Intershop PWA project is configured to work against a publicly available Intershop Commerce Management server by default (see environment.model.ts
).
icmBaseURL: 'https://develop.icm.intershop.de',
To configure your PWA project to use an own default ICM server, set the icmBaseURL
in your projects environment.model.ts
accordingly.
In the same way, the default icmChannel
configurations need to be adapted in your theme-specific environment.<THEME>.ts
files.
When adding new functionality, it is better to encapsulate it in new components that handle every aspect of the customization and just use them in existing templates.
That way the modifications on existing code are most often kept to a single line change only.
When heavily customizing existing components it is better to copy components and change all references.
If 20 % of the component has to be changed, it is already a good idea to duplicate it.
That way, incoming changes will not affect your customizations.
Typical hot-spots where copying is a good idea are header-related or product-detail-page-related customizations.
The customized webpack build supports replacing any file with an environment suffix in front of the file extension.
If you, for example, want to customize the template product-detail.component.html
, put your customized content in the parallel file product-detail.component.theme.html
and run a build with --configuration=theme
.
Then this overridden component template will be swapped in.
This also works for multiple configurations: product-detail.component.foo.bar.baz.html
will be active for configurations foo
, bar
and baz
, but not for foobar
.
There is also a virtual theme all
that can be used to always override a specific file.
However, the all
theme cannot be used in combination with other themes.
If a theme-specific override is available next to an override for all
, the specific override will be chosen.
As an example, imagine the following files/overrides exist:
my.component.html
my.component.all.html
my.component.foo.html
my.component.bar.baz.html
foo
, the file my.component.foo.html
will be swapped in.bar
, the file my.component.bar.baz.html
will be swapped in.foobar
, the file my.component.all.html
will be swapped in.my.component.html
will not be used for any builds.You can use the override
schematic to introduce custom theme overrides:
When you want to disable code provided by Intershop, it is better to comment it out instead of deleting it.
This allows Git to merge changes more predictably since original and incoming passages are still similar to each other.
Commenting out should be done in the form of block comments starting a line above and ending in an additional line below the code.
Use <!--
and -->
for HTML and /*
and */
for SCSS and TypeScript.
Some of the provided components supply configuration parameters which can be used to handle customization in an easy way (i.e. disabling features or switching between them).
When adding new independent features to the PWA, it might be a good idea to add an extension first.
Use the provided schematics ng g extension <name>
to scaffold the extension.
Adding all related pages, components, models, and services here is less intrusive than adding them to the existing folder structure.
Add additional artifacts to extensions by supplying the --extension
flag to schematics calls.
When adding new fields to PWA data models, add them to interfaces and map them as early as possible in mapper classes to model classes.
That way, the data can be readily used on templates.
Improving and parsing improper data too late could lead to more modifications on components and templates which will be harder to upgrade later on.
Adding new data to the state should always almost exclusively be done by adding new stores in store groups.
Add one with ng g store-group <group>
and then add consecutive stores with ng g store --feature <group> <store>
.
Keep modifications to the existing store as little as possible.
As NgRx is loosely coupled by nature, you can deactivate effects by commenting out the @Effect
decorator.
When modifying components it is most likely that related test cases will fail.
If possible, use the Jest update feature update snapshots when adapting test cases.
When you upgrade the PWA to a new version, those snapshots will most likely have merge conflicts in them.
Here you can just accept either modification and update the test snapshots.
You can disable creating component tests by adding the following snippet to your angular.json
:
"projects": {
"intershop-pwa": {
"schematics": {
"intershop-schematics:component": {
"skipTests": true
}
},
Changing the styling by applying changes to SCSS files should be done in the custom theme folder src/styles/themes/<theme-prefix>
.
This folder is created when adding a new theme, see Start Customization.
There are two approaches to apply a theme-specific styling:
*.scss
files you need to change to your themes folder and adjust the file references. All files which are not overwritten in your theme will be taken from the standard and all changes and bugfixes in these files when migrating the PWA will be applied and used in your project.*.scss
files to your themes folder and adjust the file references. All standard changes and bugfixes to *.scss
files will not be applied to your theme during a PWA migration.Just putting a theme override file next to the original file in the src/styles
folder will not lead to the expected results.
The lookup starts with the file style.scss
in the theme-specific folder.
Note
You should
*.scss
files in src/styles
and only apply style changes in your theme folder by copying files into the this folder and adjusting file referencesWhen styling is done on component level, all styling is encapsulated to exactly this component (default behavior).
On component level, theme-specific overrides for .scss
files work as expected.
You can re-use variables from the global styling on component level by importing only the styling file that defines the theme variables, e.g.,
@import 'variables';
Note
Be aware that Visual Studio Code will not resolve all import references correctly but it still works in the build PWA version.
Note
For bundled styles optimization PurgeCSS is used. Please read the additional documentation regarding the usage and configuration of PurgeCSS in the Intershop PWA.
To add static assets (images, favicon, manifest file), create a theme-specific folder in src/assets/themes/<theme-prefix>
and adjust the theme-specific references in the *.scss
files accordingly.
The index.html
does not support theme-specific overrides, see Theme Specific Overrides.
For this file, any theme-specific differences are handled via theme.service.ts.
When updating dependencies and resolving conflicts inside of package-lock.json
, always accept Intershop's changes first.
After that run npm install
to regenerate the file.
We currently do not support specific adaptions for customizing Cypress tests in projects.
In theory the customized PWA project can re-use our Page Objects without much adaptions if the customized PWA also uses the same selectors for CSS classes and data-testing-ids.
Specs should be copied and adapted for the project to use correct demo data.
When executing tests, the test itself requires an appropriate demo server to be launched before the run.
We currently use patterns in the test name to determine the channel for which the test can be run.
For example, login-user.b2b.b2c.e2e-spec.ts
can be run for the inSPIRED B2C and B2B channels.
The same system can be adopted for customization projects.
Importing changes of new releases is done with Git tooling.
If you stick to the guidelines in this chapter, the process of updating should run without major problems.
Also remember to use npm install
after importing a change that modified the package.json
and package-lock.json
and run tests and linting in the process.
Reading through the migrations.md and the CHANGELOG.md - here especially the 'Breaking Changes' section - should be the first step before any migration.
The first step for any migration is to add the Intershop PWA GitHub repository as an additional remote if this is not already the case.
git remote add intershop https://github.com/intershop/intershop-pwa.git
For importing changes from the current release, you can use different approaches:
For the range git cherry-pick
approach one needs to create a new branch based on the current project's main development branch, naming it, for example, migration_to_1.1
.
git checkout -b migration_to_1.1
Now the Git commits of the new Intershop PWA release will be cherry picked into this migration branch.
For this, one needs to provide the wanted commit range which should be possible by using the Intershop PWA version tags, e.g., 1.0.0
to 1.1.0
(since the end tag is a merge commit, it will lead to an error at the end of the cherry pick; to prevent this, only the commits up to the second parent should be used with ^2
).
If there are any problems with the tags, using the specific commit SHAs should always work.
git cherry-pick 1.0.0..1.1.0^2
Now each commit of the new Intershop PWA release is applied to the custom project context.
Thus, if any merge conflicts arise, this will be within the specific Intershop PWA commit context and should be mergeable with the information and diff provided for this commit in the GitHub repository.
If merge conflicts need to be resolved, it is advisable to disable any pre-commit hooks during the migration.
For this purpose, set HUSKY=0
as environment variable.
After successfully going through the range cherry pick (with git commit
and git cherry-pick --continue
after each resolved merge conflict), an npm install
will probably be required and you need to check whether the project code still works as expected.
Starting the server or npm run check
are good basic tests for that.
For the git rebase --onto
approach you need to create a new branch based on the release tag of the Intershop PWA you want to migrate to, naming it, for example, migration_to_1.1
.
git checkout -b migration_to_1.1 1.1.0
Now the branch with the Git commits of the new Intershop PWA release will be rebased onto the current project's main development branch.
To do so, you need to provide the branch name of the target branch to rebase onto.
In addition, a commit is needed where the current migration branch should be "cut off".
This is usually the current version tag of the Intershop PWA used in the custom project, e.g., 1.0.0
.
If there are any problems with the tag, using the specific commit SHAs should always work.
git rebase --onto develop 1.0.0
Now each commit of the new Intershop PWA release is applied to the custom project context.
Thus, if any merge conflicts arise, this will be within the specific Intershop PWA commit context and should be mergeable with the information and diff provided for this commit in the GitHub repository.
If merge conflicts need to be resolved, it is advisable to disable any pre-commit hooks during the migration.
For this purpose, set HUSKY=0
as environment variable.
After successfully going through the rebase onto (with git rebase --continue
after each resolved merge conflict), an npm install
will probably be required and you need to check whether the project code still works as expected.
Starting the server or npm run check
are good basic tests for that.
This is also a possible way to migrate your custom project to the latest version of the Intershop PWA, but you will have to resolve all potentially appearing conflicts at once and without the specific commit context.
Just add the Intershop PWA GitHub repository as a second remote in your project and git merge
the release branch.
The Intershop PWA project is configured to follow consistent formatting rules.
For a better overview of relevant changes and less merge efforts it is advised to adhere to these rules during project development as well.
For this, you need to configure your IDE accordingly.
The fitting Visual Studio Code configuration is part of the project.
The Intershop PWA project configures and contains a set of linting rules that also aim to ensure a consistent code style and are intended to prevent any coding patterns that are considered problematic.
It is advised to follow these rules in customer projects as well.
If some rules are actually unwanted in a project, it is best to disable these configurations but to keep all other checks intact and to address any violations.
Keep the unit tests running and write new ones.
This will also help ensuring not to break any existing functionality after a migration.
npm run check
should run through successfully on local development environments after a feature or bug fix or migration has finished.
This check can be left to a CI pipeline as well but the task should be configured in a way that fits the requirements of the project.