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

Upgrade to Angular 8 #11

Closed
wants to merge 12 commits into from
Closed

Conversation

artar94
Copy link

@artar94 artar94 commented Oct 27, 2019

  • Use one project structure for development and publishing.
    In the demo application, FeedbackModule is imported directly from ../../projects/ng-feedback/src/public_api to facilitate debugging and development.
  • html2canvas requirement as peerDependencies. Documentation updated.
  • Upgrade to Angular 8.
  • Upgrade to core-js 3.
  • Imports are given about the standard form, excess removed.
  • Added script to copy README and LICENSE to distribution.

Thanks so much for a great library!

@adiagrwl94
Copy link

If this PR supports backward angular versions (7 and 6) compatibility too, could this be merged - In the current version (master) - it doesn't seem to work with my Angular 8 project.

@artar94
Copy link
Author

artar94 commented Dec 23, 2019

Can you give me some details about this?
This works in my project on Angular 8, also I tested it on Stackblitz.
Angular 8: https://stackblitz.com/edit/github-wldzic
Angular 7: https://stackblitz.com/edit/github-wldzic-rxvgqs
Angular 6: https://stackblitz.com/edit/github-wldzic-9k26oz
Unfortunately, images do not work on Stackblitz, and images from other domains do not work without useCORS.

The only compatibility-breaking change is the html2canvas requirement as peerDependency. That is, html2canvas now needs to be installed explicitly in the main project. The installation line in README has been changed.

@artar94 artar94 closed this Jul 13, 2020
@artar94 artar94 deleted the feat/angular8 branch July 13, 2020 22:01
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.

2 participants