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

(deps) Upgrade Angular to 13.0.2 #1070

Closed
wants to merge 7 commits into from
Closed

(deps) Upgrade Angular to 13.0.2 #1070

wants to merge 7 commits into from

Conversation

siovene
Copy link
Collaborator

@siovene siovene commented Dec 12, 2021

No description provided.

@siovene
Copy link
Collaborator Author

siovene commented Dec 12, 2021

@satanTime would you be able to help and figure out why most test don't work anymore after trying to upgrade Angular from 12 to 13?

The app runs fine (all Cypress tests are green) so it must be a configuration setting.

The issue is with ngrx store, I believe, because every time I do this in a test:

import { MockStore } from "@ngrx/store/testing";

const store: MockStore = TestBed.inject(MockStore);

I get the following error:

Error: Importing TimeagoModule which does not have a ɵmod property

(TimeagoModule is something I use, but I don't understand why it would be connected to ngrx. Additionally, when I run the app, timeago works fine)

Thank you in advance!

@satanTime
Copy link
Contributor

Hi @siovene,

I hope you are doing great.

I think it belongs to changes in angular 13.0.3. Currently I'm investigating that, but looks like there is not an easy fix, and I need some time to publish a proper on. Hopefully asap.

Please check that issue on GitHub: help-me-mom/ng-mocks#1427, maybe its comments'd help until the fix is here.

@siovene
Copy link
Collaborator Author

siovene commented Dec 12, 2021

@satanTime ok thanks, I'll wait for that fix. Thank you very much!

@siovene
Copy link
Collaborator Author

siovene commented Dec 20, 2021

@satanTime

I think it belongs to changes in angular 13.0.3. Currently I'm investigating that, but looks like there is not an easy fix, and I need some time to publish a proper on. Hopefully asap.

But this PR is for 13.0.2. I'm reading on the issue page in your repo that you recommend staying with 13.0.2 until you can release a fix, but I'm not trying to upgrade above 13.0.2. I'm trying from 12 to 13.0.2.

Any other comments?

Thank you very much!

@satanTime
Copy link
Contributor

Hi @siovene,

You are right. With <=13.0.2, I expect that the latest ng-mocks will work.

If it still fails here, I'll take a deeper look at the upcoming weekend.

# Conflicts:
#	angular.json
#	package-lock.json
@siovene
Copy link
Collaborator Author

siovene commented Dec 23, 2021

@satanTime just FYI, I don't think this problem is related to help-me-mom/ng-mocks#1427. Could you take a look when you have time? I understand that I will need to wait after you've addressed help-me-mom/ng-mocks#1427, no problem :) Thanks!

@satanTime
Copy link
Contributor

Hi @siovene,

I've checked the failure.
If you have meant Importing FormlySelectModule which does not have a ɵmod property,
then right, we need to wait for a proper fix.

The issue here is that https://github.com/ngx-formly/ngx-formly has been built with the latest Angular 13 and already uses the new definition of declarations.

@siovene
Copy link
Collaborator Author

siovene commented Dec 25, 2021

@satanTime thanks for checking! Turns out there were/are multiple dependencies that are not ready for Angular 13 yet :(

@siovene
Copy link
Collaborator Author

siovene commented Dec 25, 2021

Blocked by Mawi137/ngx-image-cropper#470
Blocked by ihym/ngx-timeago#54

@siovene siovene closed this Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants