Skip to content
This repository was archived by the owner on Mar 23, 2022. It is now read-only.

Add trade event and fix name #99

Merged
merged 2 commits into from
Dec 27, 2021
Merged

Add trade event and fix name #99

merged 2 commits into from
Dec 27, 2021

Conversation

zh4ngx
Copy link
Contributor

@zh4ngx zh4ngx commented Nov 23, 2021

Add trade event based on docs and replace missing key with "name"

This was referenced Nov 23, 2021
Open
Open
cruales
cruales suggested changes Dec 27, 2021
View reviewed changes
Copy link
Contributor

@cruales cruales left a comment

Choose a reason for hiding this comment

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

@zhangbanger thanks for adding this and opening the PR! If you could just accept the change I proposed then I can go ahead and approve and merge it.

@zh4ngx @cruales
Update assets/openapi.yaml
b16e4cd 
Remove pattern

Co-authored-by: Christie Ruales <[email protected]>
@cruales cruales merged commit 0715036 into alpacahq:master Dec 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@cruales cruales cruales approved these changes

@gsmadi gsmadi Awaiting requested review from gsmadi

@mahmoudksherif mahmoudksherif Awaiting requested review from mahmoudksherif

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zh4ngx @cruales