Initial design rationale doc #501
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ericanderson
force-pushed
the
ea.design-rationale
branch
from
f61f56b to
7baaf18
Compare
ssanjay1
reviewed
Jul 22, 2024
|
|
||
| ### Discoverability | ||
|
|
||
| In v1, you might take a constructed client and say `client.ontology.objects.Car`, getting intellisense along the way. This style of discovery breaks tree shaking though so in v2, you get the discoverability from the import `import * as MyOSDK from "@my/osdk"`. Decent tree shaking tools will convert this to only importing what is needed while the IDE will present you with all the information. |
There was a problem hiding this comment.
Should we maybe add that hitting ctrl-space will give you more intellisense suggestions as well? I wasn't actually aware of this myself until someone pointed it out to me, though I'm not sure if it would make sense to add that in the context of a design doc
ssanjay1
approved these changes
Jul 23, 2024
ericanderson
force-pushed
the
ea.design-rationale
branch
from
7baaf18 to
073c9c2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.