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

feat: add connection param for snowhouse #1231

Merged
merged 5 commits into from
Sep 22, 2022
Merged

Conversation

sfc-gh-swinkler
Copy link
Collaborator

@sfc-gh-swinkler sfc-gh-swinkler commented Sep 22, 2022

Adding in a "Application" connection param which can be used to determine where requests are coming from.

Test Plan

  • acceptance tests

References

@sfc-gh-swinkler sfc-gh-swinkler changed the title add connection param for snowhouse feat: add connection param for snowhouse Sep 22, 2022
@github-actions
Copy link

Integration tests failure for dba287940269a9b31877662815a855fb8b1db727

@github-actions
Copy link

Integration tests failure for 2a914111e739a601217379aa63fdd4fb72cfdc29

@github-actions
Copy link

Integration tests failure for 4608fc371836f0a77665371d96ec9c6365b315bd

@github-actions
Copy link

Integration tests failure for 7aef991b709db0ecd2eafeb8b68a1bedc413f48a

@github-actions
Copy link

Integration tests success for ae21c4b66db4ebae79960cafd1d05d9cb00a9b8d

Copy link
Contributor

@sfc-gh-jlove sfc-gh-jlove left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants