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

Add implementation of getGoogleAnalyticsClientId #7158

Merged
merged 24 commits into from
Apr 18, 2023
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7071d2b
Add initial implementation of getGoogleAnalyticsClientId
dwyfrequency Mar 26, 2023
156137e
Update docs devsite
dwyfrequency Mar 26, 2023
510a89a
Add checkset
dwyfrequency Mar 26, 2023
41ef1bf
Update changeset description
dwyfrequency Mar 27, 2023
8f6ac07
Add link to client_id in docstring
dwyfrequency Mar 31, 2023
3e401cb
Update gtagWrapper to take variable number of args for potential fall…
dwyfrequency Apr 8, 2023
4c93e53
Add API test for getGoogleAnalyticsClientId
dwyfrequency Apr 10, 2023
c4b9792
Move API functionality to internal function
dwyfrequency Apr 11, 2023
54d83aa
Update docs for devsite
dwyfrequency Apr 11, 2023
2ecd32d
Removed unused function in test
dwyfrequency Apr 11, 2023
9f5cfb5
Update public api with async keyword
dwyfrequency Apr 11, 2023
c6d4109
Remove comment
dwyfrequency Apr 12, 2023
5516b49
Update doc string
dwyfrequency Apr 12, 2023
6eef67c
Update grammar of changeset
dwyfrequency Apr 12, 2023
eca9407
Remove console.log
dwyfrequency Apr 12, 2023
0e3b900
Update variable name from targetId to measurementId
dwyfrequency Apr 12, 2023
0ec1f27
Removed check for blank measurementId
dwyfrequency Apr 12, 2023
a585a16
Add ERROR_FACTORY for promise rejection
dwyfrequency Apr 13, 2023
953e499
Change fieldName from clientId
dwyfrequency Apr 13, 2023
7149472
Update AnalyticsError.NO_CLIENT_ID message
dwyfrequency Apr 13, 2023
93ed844
Merge branch 'master' into jd-getclientid-analytics-v2
dwyfrequency Apr 14, 2023
8332162
Testing adding comment
dwyfrequency Apr 18, 2023
888aba4
remove test comment
dwyfrequency Apr 18, 2023
9357f51
Remove comments
dwyfrequency Apr 18, 2023
File filter

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Remove comments
  • Loading branch information
dwyfrequency committed Apr 18, 2023
commit 9357f51218c4fa206b7932b23aa7a2d219399560
2 changes: 0 additions & 2 deletions packages/analytics/src/functions.ts
Original file line number Diff line number Diff line change
Expand Up @@ -142,8 +142,6 @@ export async function setUserProperties(
* Retrieves a unique Google Analytics identifier for the web client.
* See {@link https://developers.google.com/analytics/devguides/collection/ga4/reference/config#client_id | client_id}.
*
* @public
*
* @param gtagFunction Wrapped gtag function that waits for fid to be set before sending an event
*/
export async function internalGetGoogleAnalyticsClientId(
Expand Down
1 change: 0 additions & 1 deletion packages/analytics/src/helpers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,6 @@ function wrapGtag(
);
} else if (command === GtagCommand.CONSENT) {
const [gtagParams] = args;
// If CONFIG, second arg must be measurementId.
gtagCore(GtagCommand.CONSENT, 'update', gtagParams as ConsentSettings);
} else if (command === GtagCommand.GET) {
dwyfrequency marked this conversation as resolved.
Show resolved Hide resolved
const [measurementId, fieldName, callback] = args;
Expand Down