Skip to content

docs: allow custom package version for example zip file dependencies - #31198

Closed
brandonroberts wants to merge 1 commit into
angular:masterfrom
brandonroberts:docs-in-memory-8-x-ivy
Closed

brandonroberts wants to merge 1 commit into
angular:masterfrom
brandonroberts:docs-in-memory-8-x-ivy

Conversation

@brandonroberts

Copy link
Copy Markdown
Contributor

This change allows you to specify a version that is included in the zip
files that users download from the guides. This is a temporary workaround
due to the incompatibility of angular-in-memory-web-api with Ivy.

The decision is to leave angular-in-memory-web-api as-is until the
compatibility issue is resolved. This allows users to use the npm
version instead of the custom build we use for Ivy support with the examples.

Also adds notes on how to run the E2E tests for the docs examples with Ivy.

Fixes #28603

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • angular.io application / infrastructure changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@brandonroberts
brandonroberts requested a review from a team June 21, 2019 19:46
@mary-poppins

Copy link
Copy Markdown

You can preview ff73442 at https://pr31198-ff73442.ngbuilds.io/.

This change allows you to specify a version that is included in the zip
files that users download from the guides. This is a temporary workaround
due to the incompatibility of angular-in-memory-web-api with Ivy.

The decision is to leave angular-in-memory-web-api as-is until the
compatibility issue is resolved. This allows users to use the npm
version instead of the custom build we use for Ivy support with the examples.

Also adds notes on how to run the E2E tests for the docs examples with Ivy.

Fixes angular#28603
@brandonroberts
brandonroberts force-pushed the docs-in-memory-8-x-ivy branch from ff73442 to 00ff66e Compare June 24, 2019 15:19
@brandonroberts brandonroberts added comp: examples effort1: hours target: patch This PR is targeted for the next patch release risk: low feature Label used to distinguish feature request from other issues labels Jun 24, 2019
@ngbot ngbot Bot modified the milestone: Backlog Jun 24, 2019
@mary-poppins

Copy link
Copy Markdown

You can preview 00ff66e at https://pr31198-00ff66e.ngbuilds.io/.

@brandonroberts
brandonroberts requested a review from gkalpak June 25, 2019 19:08
@brandonroberts

Copy link
Copy Markdown
Contributor Author

No longer needed due to #30591. A separate PR will need to be opened to remove the custom build from the examples shared package.json for angular-in-memory-web-api.

@angular-automatic-lock-bot

Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Sep 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes effort1: hours feature Label used to distinguish feature request from other issues risk: low target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tour of Heroes doesn't compile with Ivy and angular-in-memory-web-api

3 participants