Skip to content
This repository was archived by the owner on May 29, 2025. It is now read-only.

Update trades summary script #31

Merged
merged 4 commits into from
Sep 29, 2023
Merged

Update trades summary script #31

merged 4 commits into from
Sep 29, 2023

Conversation

@jmoreira-valory
Copy link
Collaborator

@jmoreira-valory jmoreira-valory commented Sep 28, 2023

Update trades summary script:

  • Groups trades per market.
  • Adds CLI flags --from-date and --to-date to filter trades by trade creation date.
  • Refines the logic to determine if a trade has been redeemed.

@dagacha
Copy link
Collaborator

dagacha commented Sep 29, 2023

I've run and verified it works.

@DavidMinarsch DavidMinarsch merged commit 3445b56 into main Sep 29, 2023
@jmoreira-valory jmoreira-valory deleted the feat/update_trades_script_2 branch October 1, 2023 09:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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.

4 participants

@jmoreira-valory @dagacha @DavidMinarsch