You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, when creating NetworkTimestamp (NEM or Symbol) and we do not pass epochTime to methods toDatetime and fromDatetime by default is used mainnet epoch time:
A developer using SDK may expect that the testnet network will be supported 'by default'. He may not notice that it's required to pass epoch time for the test net. This could also to problems when moving code from testing (testnet) to production (mainnet).
Expected Behaviour
Support for testnet as is supported in Facades. Maybe add Network (with added epoch time) as a parameter?
problem: NetworkTimestamp is using mainnet epoch as default epoch.
This makes using it with other networks more difficult.
solution: move to_datetime and from_datetime into Network class from NetworkTimestamp.
provide network epoch to Network constructor.
issues: #243, #244
problem: NetworkTimestamp is using mainnet epoch as default epoch.
This makes using it with other networks more difficult.
solution: move to_datetime and from_datetime into Network class from NetworkTimestamp.
provide network epoch to Network constructor.
issues: #243, #244
problem: NetworkTimestamp is using mainnet epoch as default epoch.
This makes using it with other networks more difficult.
solution: move to_datetime and from_datetime into Network class from NetworkTimestamp.
provide network epoch to Network constructor.
issues: #243, #244
Current Behaviour
Currently, when creating
NetworkTimestamp
(NEM or Symbol) and we do not passepochTime
to methodstoDatetime
andfromDatetime
by default is used mainnet epoch time:https://github.com/symbol/symbol/blob/dev/sdk/javascript/src/symbol/NetworkTimestamp.js#L5
https://github.com/symbol/symbol/blob/dev/sdk/javascript/src/nem/NetworkTimestamp.js#L5
What's the problem?
A developer using SDK may expect that the testnet network will be supported 'by default'. He may not notice that it's required to pass epoch time for the test net. This could also to problems when moving code from testing (testnet) to production (mainnet).
Expected Behaviour
Support for testnet as is supported in Facades. Maybe add Network (with added epoch time) as a parameter?
The corresponding issue for Python SDK: #244
The text was updated successfully, but these errors were encountered: