Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Angular examples by linking @handsontable/angular package #9600

Merged
merged 19 commits into from Jun 23, 2022

Conversation

budnix
Copy link
Member

@budnix budnix commented Jun 22, 2022

Context

The PR is a continuation of #9590. There are some problems with Angular 12 and 13 examples. The PR fixes them by linking not only the handsontable package to the local node_modules directory but also the @handsontable/angular wrapper (with an exception for Angular 9 and 10 examples).

[skip changelog] (hotfix)

How has this been tested?

I tested the changes locally and I pushed the code to the pseudo-release branch to make sure that the GH workflow passes https://github.com/handsontable/handsontable/actions/runs/2547161971.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)

Related issue(s):

  1. Continuation of Update GH Actions dependencies #9590

Checklist:

@github-actions
Copy link

Launch the local version of documentation by running:

npm run docs:review f7c0e56118546bb5505daf1646aafd2b6d8e2e4a

@budnix budnix marked this pull request as ready for review June 23, 2022 06:55
@budnix budnix requested a review from jansiegel June 23, 2022 06:55
@budnix budnix self-assigned this Jun 23, 2022
@budnix budnix merged commit 2afb1b2 into develop Jun 23, 2022
@budnix budnix deleted the feature/fix-angular-examples branch June 23, 2022 07:02
budnix added a commit that referenced this pull request Jun 23, 2022
The commit is a continuation of #9590. There are some problems with Angular 12
and 13 examples. The PR fixes them by linking not only the handsontable package
to the local `node_modules` directory but also the `@handsontable/angular
wrapper` (with an exception for Angular 9 and 10 examples).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants