Skip to content
This repository has been archived by the owner on Jan 13, 2023. It is now read-only.

Removing transaction symbols from package namespace #199

Closed

Conversation

marko-k0
Copy link

Fix #185 . Please review and let me know if I missed something.

szepnapot and others added 30 commits September 28, 2017 11:28
Fixes replayed bundle being reversed
fix transfer = ProposedTransaction(...) naming convention doesn't follow other libs iotaledger#72
The previous name will still work, but it is deprecated and will be removed in PyOTA v2.1.
…esFromTrytes

Renamed 'trytes' codec to 'trytes_ascii'.
@todofixthis
Copy link
Contributor

Hey @redondo-mk thanks so much for working on this! I will take a look this weekend 😺

@todofixthis
Copy link
Contributor

Looks great, thanks @redondo-mk !!

There's a few remaining references hiding out in the examples and docs directories; would you be willing to have a look when you get a chance?

@marko-k0 marko-k0 closed this Jul 25, 2018
@marko-k0 marko-k0 reopened this Jul 25, 2018
@marko-k0
Copy link
Author

Sorry that I missed that - tests didn't save me there. I belive it should be good now.

@todofixthis
Copy link
Contributor

jeje no worries. Thanks @redondo-mk !! Gonna give this a final once-over, and then it should be ready to merge.

@marko-k0 marko-k0 force-pushed the 185_symbols_in_transaction_init branch from 0154c53 to 4a1181e Compare July 28, 2018 17:59
@marko-k0 marko-k0 closed this Jul 28, 2018
@marko-k0
Copy link
Author

I've tried to update my email address from my first commit in this branch and I've done something wrong. I've created another branch 185_symbols_in_transaction_init2 which has changes relevant to #185. I apologize for inconvenience!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.