abacritt / angularx-social-login

Social login and authentication module for Angular 17
630 stars 387 forks source link

GoogleSigninButtonDirective has a minimum angular version of 14 #562

Closed shkarface closed 1 year ago

shkarface commented 1 year ago

When adding this library on Angular 13 (~13.3.0), I get this compilation error:

./node_modules/@abacritt/angularx-social-login/fesm2015/abacritt-angularx-social-login.mjs - Error: Module build failed (from ./node_modules/@angular-devkit/build-angular/src/babel/webpack-loader.js):    
TypeError: Cannot create property 'message' on string 'C:\Users\shkar\src\peshbrke-panel\node_modules\@abacritt\angularx-social-login\fesm2015\abacritt-angularx-social-login.mjs: This application depends upon a library published using Angular version 14.0.3, which requires Angular version 14.0.0 or newer to work correctly.
Consider upgrading your application to use a more recent version of Angular.
  423 | }
  424 | GoogleSigninButtonDirective.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "14.0.3", ngImport: i0, type: GoogleSigninButtonDirective, deps: [{ token: i0.ElementRef }, { token: SocialAuthService }], target: i0.ɵɵFactoryTarget.Directive });
> 425 | GoogleSigninButtonDirective.ɵdir = i0.ɵɵngDeclareDirective({ minVersion: "14.0.0", version: "14.0.3", type: GoogleSigninButtonDirective, selector: "asl-google-signin-button", ngImport: i0 });     
      |                                    ^^^^^^^^^^^^^^^9m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^9m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^  
  426 | i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "14.0.3", ngImport: i0, type: GoogleSigninButtonDirective, decorators: [{
  427 |             type: Directive,
  428 |             args: [{'
    at run (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transformation\index.js:37:15)
    at run.next (<anonymous>)
    at Function.transform (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transform.js:25:41)
    at transform.next (<anonymous>)
    at step (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:261:32)
    at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:273:13
    at async.call.result.err.err (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:223:11)
    at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:189:28
    at C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\gensync-utils\async.js:74:7     
    at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:113:33

Error: node_modules/@abacritt/angularx-social-login/directives/google-signin-button.directive.d.ts:7:18 - error TS2314: Generic type 'ɵɵDirectiveDeclaration' requires 6 type argument(s).

7     static ɵdir: i0.ɵɵDirectiveDeclaration<GoogleSigninButtonDirective, "asl-google-signin-button", never, {}, {}, never, never, false>;

As seen from the logs, it looks like GoogleSigninButtonDirective requires a minimum angular version of 14

MartiboDev commented 1 year ago

Getting the same issue on angular 13

proselin commented 1 year ago
....
429 |  GoogleSigninButtonDirective.ɵdir = i0.ɵɵngDeclareDirective({ minVersion: "14.0.0", version: "14.1.0", type: GoogleSigninButtonDirective, selector: "asl-google-signin-button", inputs: { type: "type", size: "size" }, ngImport: i0 });
....
465 | SocialLoginModule.ɵmod = i0.ɵɵngDeclareNgModule({ minVersion: "14.0.0", version: "14.1.0", ngImport: i0, type: SocialLoginModule, declarations: [GoogleSigninButtonDirective], imports: [CommonModule], exports: [GoogleSigninButtonDirective] });

I found it in fesm 2015, hope they fix this issue

shkarface commented 1 year ago

Any fixes for this?

shkarface commented 1 year ago

According to the angular docs, libraries must be compiled in partial mode so that it's not tied to a specific angular version.

@Heatmanofurioso can you please take a look at this?

See: https://angular.io/guide/angular-package-format#partial-compilation

Heatmanofurioso commented 1 year ago

You can always open up a PR with that change. @shkarface I want to encourage people to help maintain this library.. I'm not it's original developer, I'm only someone who offered to help maintain it, and my time is kinda limited these days.

Also, which version of the library are you using? Because I believe we should be supporting V13 in the latest version

shkarface commented 1 year ago

Thank you @Heatmanofurioso for the suggestion, I might just do that.

I'm used v1.2.2 in my original test, and I tried the latest v1.2.3 as well and the issue is still present.

Heatmanofurioso commented 1 year ago

Hi @shkarface Can you test with V1.2.4, please?

shkarface commented 1 year ago

I still have the same issue, logs are:

./node_modules/@abacritt/angularx-social-login/fesm2015/abacritt-angularx-social-login.mjs - Error: Module build failed (from ./node_modules/@angular-devkit/build-angular/src/babel/webpack-loader.js): TypeError: Cannot create property 'message' on string 'C:\Users\shkar\src\peshbrke-panel\node_modules\@abacritt\angularx-social-login\fesm2015\abacritt-angularx-social-login.mjs: This application depends upon a library published using Angular version 14.2.0, which requires Angular version 14.0.0 or newer to work correctly. Consider upgrading your application to use a more recent version of Angular. 459 | } 460 | GoogleSigninButtonDirective.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "14.2.0", ngImport: i0, type: GoogleSigninButtonDirective, deps: [{ token: i0.ElementRef }, { token: SocialAuthService }], target: i0.ɵɵFactoryTarget.Directive });

461 | GoogleSigninButtonDirective.ɵdir = i0.ɵɵngDeclareDirective({ minVersion: "14.0.0", version: "14.2.0", type: GoogleSigninButtonDirective, selector: "asl-google-signin-button", inputs: { type: "type", size: "size", text: "text", shape: "shape", theme: "theme", logo_alignment: "logo_alignment", width: "width", locale: "locale" }m, ngImport: i0 }); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^ 462 | i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "14.2.0", ngImport: i0, type: GoogleSigninButtonDirective, decorators: [{ 463 | type: Directive, 464 | args: [{' at run (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transformation\index.js:37:15)
at run.next () at Function.transform (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transform.js:25:41)
at transform.next () at step (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:261:32) at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:273:13 at async.call.result.err.err (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:223:11)
at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:37:40

Error: node_modules/@abacritt/angularx-social-login/directives/google-signin-button.directive.d.ts:15:18 - error TS2314: Generic type 'ɵɵDirectiveDeclaration' requires 6 type argument(s).

15 static ɵdir: i0.ɵɵDirectiveDeclaration<GoogleSigninButtonDirective, "asl-google-signin-button", never, { "type": "type"; "size": "size"; "text": "text"; "shape": "shape"; "theme": "theme"; "logo_alignment": "logo_alignment"; "width": "width"; "locale": "locale"; }, {}, never, never, false>;


** Angular Live Development Server is listening on localhost:4200, open your browser on http://localhost:4200/ **

× Failed to compile.
✔ Browser application bundle generation complete.

8 unchanged chunks

Build at: 2022-08-29T18:30:51.819Z - Hash: a5617ed048f38797 - Time: 989ms

./node_modules/@abacritt/angularx-social-login/fesm2015/abacritt-angularx-social-login.mjs - Error: Module build failed (from ./node_modules/@angular-devkit/build-angular/src/babel/webpack-loader.js):
TypeError: Cannot create property 'message' on string 'C:\Users\shkar\src\peshbrke-panel\node_modules\@abacritt\angularx-social-login\fesm2015\abacritt-angularx-social-login.mjs: This application depends upon a library published using Angular version 14.2.0, which requires Angular version 14.0.0 or newer to work correctly.
Consider upgrading your application to use a more recent version of Angular.
  459 | }
  460 | GoogleSigninButtonDirective.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "14.2.0", ngImport: i0, type: GoogleSigninButtonDirective, deps: [{ token: i0.ElementRef }, { token: SocialAuthService }], target: i0.ɵɵFactoryTarget.Directive });
> 461 | GoogleSigninButtonDirective.ɵdir = i0.ɵɵngDeclareDirective({ minVersion: "14.0.0", version: "14.2.0", type: GoogleSigninButtonDirective, selector: "asl-google-signin-button", inputs: { type: "type", size: "size", text: "text", shape: "shape", theme: "theme", logo_alignment: "logo_alignment", width: "width", locale: "locale" }m, ngImport: i0 });
      |                                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^
  462 | i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "14.2.0", ngImport: i0, type: GoogleSigninButtonDirective, decorators: [{
  463 |             type: Directive,
  464 |             args: [{'
    at run (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transformation\index.js:37:15)      
    at run.next (<anonymous>)
    at Function.transform (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transform.js:25:41)  
    at transform.next (<anonymous>)
    at step (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:261:32)
    at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:273:13
    at async.call.result.err.err (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:223:11)      
    at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:37:40

Error: node_modules/@abacritt/angularx-social-login/directives/google-signin-button.directive.d.ts:15:18 - error TS2314: Generic type 'ɵɵDirectiveDeclaration' requires 6 type argument(s).

15     static ɵdir: i0.ɵɵDirectiveDeclaration<GoogleSigninButtonDirective, "asl-google-signin-button", never, { "type": "type"; "size": "size"; "text": "text"; "shape": "shape"; "theme": "theme"; "logo_alignment": "logo_alignment"; "width": "width"; "locale": "locale"; }, {}, never, never, false>;

× Failed to compile. ✔ Browser application bundle generation complete.

8 unchanged chunks

Build at: 2022-08-29T18:30:52.712Z - Hash: a5617ed048f38797 - Time: 704ms

./node_modules/@abacritt/angularx-social-login/fesm2015/abacritt-angularx-social-login.mjs - Error: Module build failed (from ./node_modules/@angular-devkit/build-angular/src/babel/webpack-loader.js): TypeError: Cannot create property 'message' on string 'C:\Users\shkar\src\peshbrke-panel\node_modules\@abacritt\angularx-social-login\fesm2015\abacritt-angularx-social-login.mjs: This application depends upon a library published using Angular version 14.2.0, which requires Angular version 14.0.0 or newer to work correctly. Consider upgrading your application to use a more recent version of Angular. 459 | } 460 | GoogleSigninButtonDirective.ɵfac = i0.ɵɵngDeclareFactory({ minVersion: "12.0.0", version: "14.2.0", ngImport: i0, type: GoogleSigninButtonDirective, deps: [{ token: i0.ElementRef }, { token: SocialAuthService }], target: i0.ɵɵFactoryTarget.Directive });

461 | GoogleSigninButtonDirective.ɵdir = i0.ɵɵngDeclareDirective({ minVersion: "14.0.0", version: "14.2.0", type: GoogleSigninButtonDirective, selector: "asl-google-signin-button", inputs: { type: "type", size: "size", text: "text", shape: "shape", theme: "theme", logo_alignment: "logo_alignment", width: "width", locale: "locale" }m, ngImport: i0 }); | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^m^^^^^^^^^^^^^^ 462 | i0.ɵɵngDeclareClassMetadata({ minVersion: "12.0.0", version: "14.2.0", ngImport: i0, type: GoogleSigninButtonDirective, decorators: [{ 463 | type: Directive, 464 | args: [{' at run (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transformation\index.js:37:15)
at run.next () at Function.transform (C:\Users\shkar\src\peshbrke-panel\node_modules\@babel\core\lib\transform.js:25:41)
at transform.next () at step (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:261:32) at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:273:13 at async.call.result.err.err (C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:223:11)
at C:\Users\shkar\src\peshbrke-panel\node_modules\gensync\index.js:37:40

Error: node_modules/@abacritt/angularx-social-login/directives/google-signin-button.directive.d.ts:15:18 - error TS2314: Generic type 'ɵɵDirectiveDeclaration' requires 6 type argument(s).

15 static ɵdir: i0.ɵɵDirectiveDeclaration<GoogleSigninButtonDirective, "asl-google-signin-button", never, { "type": "type"; "size": "size"; "text": "text"; "shape": "shape"; "theme": "theme"; "logo_alignment": "logo_alignment"; "width": "width"; "locale": "locale"; }, {}, never, never, false>;



× Failed to compile.
stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.