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.

Sign up for GitHub

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

Jump to bottom

11/29/24 Port Order Trades components #1032

Draft
wants to merge 45 commits into
base: main
Choose a base branch
from
Draft

11/29/24 Port Order Trades components #1032

wants to merge 45 commits into from

Conversation

hardingjam
Copy link
Contributor

Motivation

Solution

Checks

By submitting this for review, I'm confirming I've done the following:

  • made this PR as small as possible
  • unit-tested any new functionality
  • linked any relevant issues or PRs
  • included screenshots (if this involves a front-end change)

hardingjam and others added 30 commits November 26, 2024 10:31
@hardingjam
badge and button
c5e3319
@hardingjam
move filters across
4f54d49
@hardingjam
tests passing
318ec1d
@hardingjam
add
440e238
@hardingjam
fix formatting
3dda2e9
@hardingjam
darkmode codemirror theme and move tests
06fec91
@hardingjam
BadgeActve test
d6c78a7
@hardingjam
reinstatte chart time filter
26c0cce
@hardingjam
test
abeed2a
@hardingjam
Merge branch '11/26/24-orderdetail-step-1' into 11/26/24-Order-Charts-1
31a7cfa
@hardingjam
remove dupes
ce8c672
@hardingjam
fix tests
e322f14
@hardingjam
move theme to props
7be9650
@hardingjam
move themes
793b9ab
@hardingjam
move
ef1d44e
@hardingjam
format
fce463c
@hardingjam
move orders
b2ce386
@findolor
update tests
17e47e3
@hardingjam
add trade type
0e9ff1f
@hardingjam
add trade functions
5eb25aa
@hardingjam
remove some
f0d9def
@hardingjam
show trade detail in chart
c6b9f54
@hardingjam
remove count
df5c183
@hardingjam
trade detail and count
9385ea8
@hardingjam
last test not passing
419ff4e
@hardingjam
test order count
955a5bf
@hardingjam
replace vaultId
3f86d7d
@hardingjam
add tests
e632879
@hardingjam
format
f619dbf
@hardingjam
Merge branch 'add-order-trades-list-wasm' into move-trades-components
2b7fb57
@hardingjam hardingjam changed the title 11/29/24 Por Order Trades components 11/29/24 Port Order Trades components Nov 29, 2024
@hardingjam hardingjam changed the base branch from main to add-order-trades-list-wasm November 29, 2024 13:33
@hardingjam hardingjam changed the base branch from add-order-trades-list-wasm to main November 29, 2024 13:34
@hardingjam hardingjam marked this pull request as draft November 29, 2024 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@hardingjam @findolor