Skip to content

Switch Tx to a data family. #5000

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

Open
lehins opened this issue Apr 17, 2025 · 0 comments · May be fixed by #5097
Open

Switch Tx to a data family. #5000

lehins opened this issue Apr 17, 2025 · 0 comments · May be fixed by #5097
Assignees
Labels
💳 technical-debt Issues related to technical debt we introduced

Comments

@lehins
Copy link
Collaborator

lehins commented Apr 17, 2025

Unlike #4999, Tx has changed only twice, since shelley: ShelleyEra and AlonzoEra. Therefore this ticket will likely add a little bit more boiler plate. However, it will be very useful, since it will allow us to get rid of superficial newtype wrappers around the Tx in a whole varieties of places:

During translation in ledger:

In consensus:

There is probably some more that I am missing.

@lehins lehins added the 💳 technical-debt Issues related to technical debt we introduced label Apr 17, 2025
@Soupstraw Soupstraw self-assigned this Apr 24, 2025
@Soupstraw Soupstraw mentioned this issue Apr 24, 2025
10 tasks
@Soupstraw Soupstraw linked a pull request Jun 2, 2025 that will close this issue
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💳 technical-debt Issues related to technical debt we introduced
Projects
None yet
2 participants