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

Allow Angular 5 #40

Merged
merged 4 commits into from
Aug 22, 2019
Merged

Allow Angular 5 #40

merged 4 commits into from
Aug 22, 2019

Conversation

engineering-this
Copy link
Contributor

@engineering-this engineering-this commented Jul 5, 2019

Applied change required to work with Angular 5, see #39 (comment)

Also changed all leftover Angular 2+ references so that version isn't mentioned.

Closes #39. Closes #37. Closes #42.

@engineering-this
Copy link
Contributor Author

At first I didn't think about closing the issue, so I've created branch under different name.

@f1ames f1ames self-requested a review July 12, 2019 14:51
@f1ames f1ames self-assigned this Jul 12, 2019
Copy link
Contributor

@f1ames f1ames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also update README on Angular versions compatibility (you make take a look how CKE5 does this)?

@engineering-this
Copy link
Contributor Author

I copied section from CKE5 README, our case is identical, so I don't think it is necessary to reword it in any way.

Copy link
Contributor

@f1ames f1ames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍

@Sakamotto
Copy link

When the new releasw will be in npm repository?

@f1ames
Copy link
Contributor

f1ames commented Nov 13, 2019

@Sakamotto next week, between 19-21.11. It will be 1.0.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants