Skip to content

feat: wire amplitude v2 autocapture toggles#3091

Open
rudder-devbox[bot] wants to merge 8 commits into
developfrom
harness/SDK-5014-feat-wire-amplitude-v2-autocapt-da5ae275
Open

feat: wire amplitude v2 autocapture toggles#3091
rudder-devbox[bot] wants to merge 8 commits into
developfrom
harness/SDK-5014-feat-wire-amplitude-v2-autocapt-da5ae275

chore: address copilot review comments

7ac3a77
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Jul 8, 2026 in 52s

Annotations

Check warning on line 145 in packages/analytics-js-integrations/__tests__/integrations/Amplitude/util.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Prefer a more specific assertion instead of this generic one, e.g. "expect(result).toBeUndefined()".

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WK6as0eDFONNlEX&open=AZ8c6WK6as0eDFONNlEX&pullRequest=3091

Check failure on line 406 in packages/analytics-js-integrations/__tests__/integrations/Amplitude/browser.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case.

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WKFas0eDFONNlEU&open=AZ8c6WKFas0eDFONNlEU&pullRequest=3091

Check warning on line 124 in packages/analytics-js-integrations/src/integrations/Amplitude/browser.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WM3as0eDFONNlEb&open=AZ8c6WM3as0eDFONNlEb&pullRequest=3091

Check failure on line 509 in packages/analytics-js-integrations/__tests__/integrations/Amplitude/browser.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case.

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WKFas0eDFONNlEW&open=AZ8c6WKFas0eDFONNlEW&pullRequest=3091

Check warning on line 27 in packages/analytics-js-integrations/src/integrations/Amplitude/constants.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `export…from` to re-export `DISPLAY_NAME`.

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WNNas0eDFONNlEs&open=AZ8c6WNNas0eDFONNlEs&pullRequest=3091

Check failure on line 460 in packages/analytics-js-integrations/__tests__/integrations/Amplitude/browser.test.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Add at least one assertion to this test case.

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WKFas0eDFONNlEV&open=AZ8c6WKFas0eDFONNlEV&pullRequest=3091

Check warning on line 123 in packages/analytics-js-integrations/src/integrations/Amplitude/browser.js

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Use `.includes()`, rather than `.indexOf()`, when checking for existence.

See more on https://sonarcloud.io/project/issues?id=rudderlabs_repo_rudder-sdk-js&issues=AZ8c6WM3as0eDFONNlEa&open=AZ8c6WM3as0eDFONNlEa&pullRequest=3091