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

Compatibility with Angular 16 and Angular Ivy #15

Open
mspnr opened this issue Jul 9, 2023 · 9 comments
Open

Compatibility with Angular 16 and Angular Ivy #15

mspnr opened this issue Jul 9, 2023 · 9 comments
Labels
bug Something isn't working

Comments

@mspnr
Copy link

mspnr commented Jul 9, 2023

After upgrade from Angular 14 to 16 I've got the message:

'PixelModule' does not appear to be an NgModule class.(-996002)
pixel.module.d.ts(5, 8): This likely means that the library (ngx-pixel) which declares PixelModule is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.

I am using the latest version of the library from npm:

"ngx-pixel": "^1.1.1",

@NielsCodes
Copy link
Owner

Hey, it should support Ivy, but I haven't tested with Angular 16 yet. I'll do some testing and get back to this in the coming days.

@NielsCodes NielsCodes added the bug Something isn't working label Jul 29, 2023
@duxor
Copy link

duxor commented Sep 8, 2023

I also have the same issue. Let's make a solution during the next week 😄

@rysal32
Copy link

rysal32 commented Nov 22, 2023

Any solution so far?

@sitthinan
Copy link

I have the same problem bro, @NielsCodes do you have any updates?

@onclave
Copy link

onclave commented Jan 29, 2024

Hello @NielsCodes , first of all, thank you so much for this awesome library. However, it would be great if you could provide an update for this issue, since I am stuck at upgrading with Angular 17 for this library. Could you kindly look into this? I think all you have to do is recompile the project with Angular 17. Would you be interested in a PR to resolve this issue?

@onclave
Copy link

onclave commented Jan 29, 2024

@NielsCodes I have raised a PR ( #18 ) to update the library to Angular v17. Kindly merge the PR and make a new release, if possible. Thanks in advance!

@onclave
Copy link

onclave commented Feb 8, 2024

@NielsCodes since I did not receive any communication from you, I have gone ahead and released this same code under a different library name that supports Angular v17+ here - @bloomscorp/ngx-pixel since your library is under MIT license. I hope this helps others as well.

Also, on a different note, I hope you are doing good since I did not receive any communication from your end. I hope everything is alright on your end. And thank you, once again, for this library.

@morsagmon
Copy link

@onclave Thank you for this adjustment!
An important modification would be to add support for the eventID param Facebook is encouraging to pass, in order to fortify deduplication of events reported through both the Pixel and ConversionAPI.
This is how you're calling fbq now:
fbq('track', eventName, properties);

The eventID is a possible fourth argument :
fbq('track', eventName, properties, {eventID: 'EVENT_ID'});

Please see here:
https://developers.facebook.com/docs/marketing-api/conversions-api/deduplicate-pixel-and-server-events

I believe as more will be leveraging the strong ConversionAPI alongside the Pixel, this will be very much needed.

Thanks!
Mor

@morsagmon
Copy link

@onclave
I went ahead and forked out from your repo.
I adjusted the package to support the eventID param to Facebook.
I also updated the readme to reflect your changes and my changes.
Here:
https://www.npmjs.com/package/ngx-pixel-eventid

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants