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

Updated metric payload to include new tags #27

Merged
merged 9 commits into from
Dec 15, 2023

Conversation

cooperwilletts-cb
Copy link
Contributor

Description

Please provide a description of your PR and which issue it fixes.
If this PR is a work in progress, please prefix the title with [WIP].

Updated metric enhancer to include extra metadata with requests

Type of change

  • [] Bug fix (non-breaking change which fixes an issue)
  • [] New feature (non-breaking change which adds functionality)
  • [] Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • [] Refactor (refactoring production code or docs)
  • [] Test (adding missing tests or fixing existing tests)
  • [] Chore (updating yarn tasks etc; no production code change)
  • [] Other (please describe):

Checklist

  • [] unit tests

Update example files

If you have updated the example files, please provide a screenshot of the changes (if appropriate).

Copy link

changeset-bot bot commented Dec 15, 2023

⚠️ No Changeset found

Latest commit: 08d3ebd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@cb-heimdall cb-heimdall dismissed giuseppe-coinbase’s stale review December 15, 2023 18:14

Approved review 1784598547 from giuseppe-coinbase is now dismissed due to new commit. Re-request for approval.

README.md Outdated Show resolved Hide resolved
@giuseppe-coinbase giuseppe-coinbase merged commit 9e43ef8 into master Dec 15, 2023
4 checks passed
@giuseppe-coinbase giuseppe-coinbase deleted the willetts/add-new-tags branch December 15, 2023 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants