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

[SDK-3858] Add documentation for modular variant of library #1502

Conversation

lawrence-forooghian
Copy link
Collaborator

@lawrence-forooghian lawrence-forooghian commented Nov 14, 2023

Adds documentation (TypeDoc and in README) for the modular variant of the library. See commit messages for more details.

Resolves #1443.

@github-actions github-actions bot temporarily deployed to staging/pull/1502/features November 14, 2023 18:46 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/bundle-report November 14, 2023 18:47 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/typedoc November 14, 2023 18:47 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1442-add-typings-for-tree-shakable-version branch from 869be09 to b839715 Compare November 14, 2023 19:05
@lawrence-forooghian lawrence-forooghian force-pushed the 1443-add-documentation-for-tree-shakable-version branch from 98782c4 to 3d152b9 Compare November 14, 2023 19:05
@github-actions github-actions bot temporarily deployed to staging/pull/1502/features November 14, 2023 19:06 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/bundle-report November 14, 2023 19:06 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/typedoc November 14, 2023 19:06 Inactive
@lawrence-forooghian lawrence-forooghian changed the title 1443 add documentation for tree shakable version [SDK-3858] Add documentation for modular variant of library Nov 14, 2023
modules.d.ts Outdated Show resolved Hide resolved
modules.d.ts Outdated Show resolved Hide resolved
@lawrence-forooghian lawrence-forooghian marked this pull request as ready for review November 14, 2023 20:23
@lawrence-forooghian lawrence-forooghian force-pushed the 1443-add-documentation-for-tree-shakable-version branch from 3d152b9 to c9d881f Compare November 15, 2023 14:31
@github-actions github-actions bot temporarily deployed to staging/pull/1502/features November 15, 2023 14:31 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/typedoc November 15, 2023 14:32 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/bundle-report November 15, 2023 14:32 Inactive
@stmoreau stmoreau requested a review from ttypic November 16, 2023 13:54
Base automatically changed from 1442-add-typings-for-tree-shakable-version to integration/v2 November 21, 2023 00:18
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
@lawrence-forooghian lawrence-forooghian force-pushed the 1443-add-documentation-for-tree-shakable-version branch from c9d881f to 72ba373 Compare November 22, 2023 19:19
@github-actions github-actions bot temporarily deployed to staging/pull/1502/features November 22, 2023 19:19 Inactive
@lawrence-forooghian lawrence-forooghian force-pushed the 1443-add-documentation-for-tree-shakable-version branch from 72ba373 to d750acd Compare November 22, 2023 19:20
@github-actions github-actions bot temporarily deployed to staging/pull/1502/features November 22, 2023 19:20 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/typedoc November 22, 2023 19:21 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/1502/bundle-report November 22, 2023 19:21 Inactive
Copy link
Member

@owenpearson owenpearson left a comment

Choose a reason for hiding this comment

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

Looks excellent, nice one 👍 I've got one additional suggested tweak but feel free to ignore if you disagree. I'll approve now so you can merge when you've resolved the eslint ignore comment

README.md Outdated Show resolved Hide resolved
modules.d.ts Outdated Show resolved Hide resolved
I think the docs/ directory is something that’s meant to contain
documentation for direct consumption by users or developers, and not
fragments that need to be glued together with something else to be
useful.
I imagine there’s plenty of room for improvement here, but it’s a start.

Hopefully a tech writer will be able to have a look at it before we
release, too.

For handling TypeDoc documentation for the two different variants of the
library, I’ve resurrected the approach that we used when we had
callback-based and Promise-based variants of the library (removed in
2a2ed49).

Resolves #1443.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants