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 state & variables to GetExhangeResponse. #428

Merged
merged 4 commits into from
Nov 19, 2024

Conversation

BigBlueHat
Copy link
Collaborator

Can't track state if it's not there. 😃

exchanges.yml Outdated Show resolved Hide resolved
exchanges.yml Outdated Show resolved Hide resolved
exchanges.yml Outdated Show resolved Hide resolved
@BigBlueHat
Copy link
Collaborator Author

@dlongley let me know if you want more cleanup throughout. I'd prefer we merge this (if it's ready), though, and make an issue (or another PR) for further cleanup.

@BigBlueHat BigBlueHat requested a review from dlongley October 30, 2024 13:52
exchanges.yml Outdated Show resolved Hide resolved
@msporny msporny merged commit 5aff850 into main Nov 19, 2024
1 check passed
@msporny msporny deleted the add-missing-exchange-response-bits branch November 19, 2024 20:45
@TallTed
Copy link
Collaborator

TallTed commented Nov 19, 2024

Even though already merged, it seems worth correcting the spelling of GetExhangeResponse in the title (and initial commit) of this PR, to GetExchangeResponse.

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.

4 participants