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 siteHash and userAgent #27170

Merged
merged 4 commits into from Oct 1, 2020
Merged

Add siteHash and userAgent #27170

merged 4 commits into from Oct 1, 2020

Conversation

jamo
Copy link
Contributor

@jamo jamo commented Sep 29, 2020

No description provided.

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Sep 29, 2020
@jamo jamo requested a review from ascorbic September 29, 2020 16:25
@gatsby-cloud
Copy link

gatsby-cloud bot commented Sep 29, 2020

Gatsby Cloud Build Report

gatsby

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 19m

mfrachet
mfrachet previously approved these changes Sep 30, 2020
@LekoArts LekoArts added topic: telemetry* and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Sep 30, 2020
ascorbic
ascorbic previously approved these changes Sep 30, 2020
@jamo jamo dismissed stale reviews from ascorbic and mfrachet via 0104c85 September 30, 2020 09:44
@jamo jamo added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Sep 30, 2020
return
}
if (typeof tags.siteHash === `undefined`) {
tags.siteHash = this.siteHash
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ooh, a place in the wild where ??= is going to be useful when it lands!

@gatsbybot gatsbybot merged commit def8b37 into master Oct 1, 2020
@delete-merged-branch delete-merged-branch bot deleted the oss-telemetry-part2 branch October 1, 2020 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants