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.

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

[doc] devex roadmap #4533

Merged
merged 1 commit into from
Sep 9, 2022
Merged

Conversation

sblackshear
Copy link
Collaborator

Document upcoming changes to Sui dev experience in one place

DEVX_ROADMAP.md Outdated
@@ -0,0 +1,46 @@
# Developer Experience Roadmap

This is a list of the upcoming DevX changes in the next 30 days
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we include a start date for the 30 days? If I see this on the 28th day most of the items would be in the past already

DEVX_ROADMAP.md Outdated Show resolved Hide resolved
DEVX_ROADMAP.md Outdated

## JSON RPC
*[Gateway deprecation] Sui_syncAccountState will be deprecated and removed.
* [Gateway deprecation] Gateway’s sui_executeTransaction will be replaced by fullnode’s sui_executeTransaction which has an extra parameter that allows users to choose if a method should block or return immediately.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* [Gateway deprecation] Gateway’s sui_executeTransaction will be replaced by fullnode’s sui_executeTransaction which has an extra parameter that allows users to choose if a method should block or return immediately.
* [Gateway deprecation] Gateway’s `sui_executeTransaction` method will be replaced by fullnode’s `sui_executeTransaction` method, which has an extra parameter that allows users to choose whether a method should block or return immediately.

DEVX_ROADMAP.md Outdated
* get_events_by_recipient
* get_events_by_object
* get_events_by_timerange
* sui_eventSubscription API that enables subscribing to all transaction events, including ones that don’t generate network activity (github)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why is (github) included in this line? Should be GitHub

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this was a hyperlink in the source doc to this pull request #4288

DEVX_ROADMAP.md Outdated
* get_events_by_object
* get_events_by_timerange
* sui_eventSubscription API that enables subscribing to all transaction events, including ones that don’t generate network activity (github)
* A sui_call API that emulates transaction execution, but does not commit the same to the network, akin to eth_call in Ethereum.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* A sui_call API that emulates transaction execution, but does not commit the same to the network, akin to eth_call in Ethereum.
* A sui_call API that emulates transaction execution, but does not commit the transaction to the network, akin to eth_call in Ethereum.

DEVX_ROADMAP.md Outdated

## SDK (Typescript, Rust)
* Reliability improvements in Typescript SDK support for sui_publish API
* Support for sui_subscribeEvent in Typescript SDK
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Support for sui_subscribeEvent in Typescript SDK
* Support for `sui_subscribeEvent` in the Typescript SDK

Or it could be "Support for the sui_subscribeEvent method in the Typescript SDK."

DEVX_ROADMAP.md Outdated
* Rust SDK will be kept at parity with the Typescript SDK when feasible

# Sui Move
* Support for dynamic access of child objects to nullify the need to explicitly pass child objects in a transaction as well as to enable implementation of design patterns/collections that require pointer-like parent->child representation.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Support for dynamic access of child objects to nullify the need to explicitly pass child objects in a transaction as well as to enable implementation of design patterns/collections that require pointer-like parent->child representation.
* Support for dynamic access of child objects to nullify the need to pass child objects explicitly in a transaction and to enable implementation of design patterns/collections that require pointer-like parent->child representation.

## JSON RPC
*[Gateway deprecation] Sui_syncAccountState will be deprecated and removed.
* [Gateway deprecation] Gateway’s sui_executeTransaction will be replaced by fullnode’s sui_executeTransaction which has an extra parameter that allows users to choose if a method should block or return immediately.
* Pagination support for the following APIs.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are these all APIs or methods of the Sui API?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these are a subset that may potential return a very lengthy list and hence requires pagination.

DEVX_ROADMAP.md Outdated

# Sui Move
* Support for dynamic access of child objects to nullify the need to explicitly pass child objects in a transaction as well as to enable implementation of design patterns/collections that require pointer-like parent->child representation.
* Support for passing strings and object ID's encodings to entry functions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
* Support for passing strings and object ID's encodings to entry functions
* Support for passing strings and object ID encodings to entry functions

or encoded object IDs?

DEVX_ROADMAP.md Outdated
# Sui Move
* Support for dynamic access of child objects to nullify the need to explicitly pass child objects in a transaction as well as to enable implementation of design patterns/collections that require pointer-like parent->child representation.
* Support for passing strings and object ID's encodings to entry functions
* [done] Support for passing a vector of objects to entry functions which will open up a variety of flexible use cases wrt payments and coin management.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[done] means completed? Complete is a better word here, Done refers to food...I cooked it until the food was was done. I worked until the project was finished/completed.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if PRs are merged but not yet tagged for release we should not mark them as done.

DEVX_ROADMAP.md Outdated
# Sui Move
* Support for dynamic access of child objects to nullify the need to explicitly pass child objects in a transaction as well as to enable implementation of design patterns/collections that require pointer-like parent->child representation. ([issue](https://github.com/MystenLabs/sui/issues/4203))
* Support for passing strings and object ID's encodings to entry functions
* [done] This frequently requested feature allows passing a vector of objects to entry functions which will open up a variety of flexible use cases wrt payments and coin management.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sblackshear did you add the [done]? if they are only merged and not yet tagged for release they strictly speaking it is not usable. It may be good to leave out the [done] notation and direct people to the release notes instead.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 to Janet

Copy link
Contributor

@janetwu11 janetwu11 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding some comments inline.

DEVX_ROADMAP.md Outdated
# Sui Move
* Support for dynamic access of child objects to nullify the need to explicitly pass child objects in a transaction as well as to enable implementation of design patterns/collections that require pointer-like parent->child representation. ([issue](https://github.com/MystenLabs/sui/issues/4203))
* Support for passing strings and object ID's encodings to entry functions
* [done] This frequently requested feature allows passing a vector of objects to entry functions which will open up a variety of flexible use cases wrt payments and coin management.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: Italize wrt

@sblackshear sblackshear force-pushed the devex_roadmap branch 2 times, most recently from 8e91493 to 6a6bea5 Compare September 9, 2022 21:43
DEVX_ROADMAP.md Outdated
* `get_events_by_object`
* `get_events_by_timerange`
* A `sui_call` API that emulates transaction execution, but does not commit the transaction to the network, akin to eth_call in Ethereum. This is useful for gas cost prediction and transaction effect simulation.
* [done] `sui_eventSubscription` API that enables subscribing to all transaction events, including ones that don’t generate network activity ([issue](https://github.com/MystenLabs/sui/pull/4288))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reference to [done]

DEVX_ROADMAP.md Outdated
# Sui Move
* Support for dynamic access of child objects to avoid explicitly passing child objects as transaction inputs and to enable implementation of design patterns/collections that require pointer-like parent->child representation. ([issue](https://github.com/MystenLabs/sui/issues/4203))
* Support for passing strings and object ID encodings to entry functions
* [done] This frequently requested feature allows passing a vector of objects to entry functions which will open up a variety of flexible use cases *wrt* payments and coin management.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reference to [done]

DEVX_ROADMAP.md Outdated
* Support for dynamic access of child objects to avoid explicitly passing child objects as transaction inputs and to enable implementation of design patterns/collections that require pointer-like parent->child representation. ([issue](https://github.com/MystenLabs/sui/issues/4203))
* Support for passing strings and object ID encodings to entry functions
* [done] This frequently requested feature allows passing a vector of objects to entry functions which will open up a variety of flexible use cases *wrt* payments and coin management.
* [done] A collection of cryptographic libraries are exposed as Move native functions to use on-chain. They can be found at:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reference to [done]

@@ -0,0 +1,50 @@
# Developer Experience Roadmap
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sblackshear @janetwu11 Does it make sense to explicitly point out breaking changes ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spoke with Janet and resolving this.

@sblackshear sblackshear merged commit a99754b into MystenLabs:main Sep 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants