fix(js) Fix next.js warnings related to firebase

[marcelfolaron]

The warnings in #3160, #928 and #935 are primarily webpack related and can be mitigated using webpack magic comments. This PR solves the firebase related warnings

Checklist (if applicable):

• PR title is following https://www.conventionalcommits.org/en/v1.0.0/
• Tested (manually, unit tested, etc.)
• Docs updated (updated docs or a docs bug required)

[schnecle]

Nice! Can you describe how you tested this change?

[marcelfolaron]

Nice! Can you describe how you tested this change?

Mainly by running the use case @mbleigh used here: #3160
Anything in particular you'd want me to test?

[marcelfolaron]

PS: Sorry apparently require cannot be used in this way and so we'd need to use import. (which I had before)

[schnecle]

Nice! Can you describe how you tested this change?

Mainly by running the use case @mbleigh used here: #3160 Anything in particular you'd want me to test?

Can you make sure that you can initialize firebase telemetry with the environment variable:
export ENABLE_FIREBASE_MONITORING=true

And that if you have both a direct call to enableFirebaseTelemetry() that overrides configuration and ENABLE_FIREBASE_MONITORING=true, the overriding configuration wins?

Docs for reference:

• Enabling firebase telemetry with environment variables
• Overriding configuration to export from local

[marcelfolaron]

Nice! Can you describe how you tested this change?

Mainly by running the use case @mbleigh used here: #3160 Anything in particular you'd want me to test?

Can you make sure that you can initialize firebase telemetry with the environment variable: export ENABLE_FIREBASE_MONITORING=true

And that if you have both a direct call to enableFirebaseTelemetry() that overrides configuration and ENABLE_FIREBASE_MONITORING=true, the overriding configuration wins?

Docs for reference:

• Enabling firebase telemetry with environment variables
• Overriding configuration to export from local

Thanks for the pointer. I can confirm that the solution with import() appears to be working with ENABLE_FIREBASE_MONITORING=true as well as enableFirebaseTelemetry() including the precedence test.

[marcelfolaron]

@schnecle let me know if there is anything else I should test or do.

[schnecle]

LGTM! Thanks for running the extra tests!

[marcelfolaron]

Thank you for pointing it out!