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

Remove Exchanges #1799

Closed
Tracked by #1583
elliedavidson opened this issue Sep 22, 2023 · 0 comments · Fixed by #2073, #2071, #2074 or #2081
Closed
Tracked by #1583

Remove Exchanges #1799

elliedavidson opened this issue Sep 22, 2023 · 0 comments · Fixed by #2073, #2071, #2074 or #2081
Assignees
@bfish713 @elliedavidson
Labels
sprint6 tech-debt vid

Comments

@elliedavidson
Copy link
Member

elliedavidson commented Sep 22, 2023 •
edited by bfish713
Loading

Goal is to remove the Exchanges trait and remove the Exchange trait
@elliedavidson elliedavidson self-assigned this Sep 22, 2023
@elliedavidson elliedavidson mentioned this issue Sep 22, 2023
Merged
@bfish713 bfish713 added the vid label Nov 15, 2023
@ggutoski ggutoski mentioned this issue Nov 16, 2023
Closed
12 tasks
@bfish713 bfish713 self-assigned this Nov 16, 2023
This was linked to pull requests Nov 16, 2023
Remove NodeImplementation Generics from Network Types #2073
Merged
Remove Message Traits #2071
Merged
Cleanup after Vote/Certificate Refactor #2074
Merged
@bfish713 bfish713 changed the title Rework Exchanges Remove Exchanges Nov 16, 2023
@bfish713 bfish713 linked a pull request Nov 17, 2023 that will close this issue
Completely Remove Exchanges and Exchange traits #2081
Merged
@bfish713 bfish713 reopened this Nov 17, 2023
@bfish713 bfish713 mentioned this issue Nov 17, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bfish713 bfish713

@elliedavidson elliedavidson

Labels
sprint6 tech-debt vid
Projects
None yet
Milestone
No milestone
2 participants
@bfish713 @elliedavidson