Skip to content
This repository has been archived by the owner on Dec 26, 2023. It is now read-only.

Icons not added to manifest if inject=false #102

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Icons not added to manifest if inject=false #102

wants to merge 2 commits into from

Commits on Jun 21, 2019

  1. add prepare to package.json to allow referencing from github

    @FlorianBruckner
    FlorianBruckner committed Jun 21, 2019
    Configuration menu
    Copy the full SHA
    e16d885 View commit details
    Browse the repository at this point in the history

  2. icons are not put into the manifest if "inject=false". This is due to… 

    … the manifest resources are processed twice when inject=false and icons have been removed from options in the previous pass.

This commit fixes the problem by assuming that when assets are present, they need not be generated a second time even if inject=false.
    @FlorianBruckner
    FlorianBruckner committed Jun 21, 2019
    Configuration menu
    Copy the full SHA
    38bf378 View commit details
    Browse the repository at this point in the history