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

xmtp-node-js #92

Closed
wants to merge 20 commits into from
Closed

xmtp-node-js #92

wants to merge 20 commits into from

Conversation

humanagent
Copy link
Contributor

@humanagent humanagent commented Dec 1, 2023 •
edited
Loading

Description

This PR unifies different repos under xmtp-node-js to showcase XMTP JS SDK features.

  • Added xmtp-bot-starter
    • Improved bot-starter typescript snippets
    • README's improvements
  • fs-persitence README
  • grpc-api-client improved README
  • redis-persistence improved README
  • bot-examples README
  • Added cli-starter
    • Completed example flow. Working successfully
    • Added test
    • All commands 
      ./xmtp init
./xmtp send 0xF8cd371Ae43e1A6a9bafBB4FD48707607D24aE43 "Hello world"
./xmtp list-messages 0xF8cd371Ae43e1A6a9bafBB4FD48707607D24aE43
./xmtp stream-all
./xmtp stream 0xF8cd371Ae43e1A6a9bafBB4FD48707607D24aE43

Pending

  • Archive cli-starter
  • Archive bot-starter
  • rename repo to xmtp-node-js

@humanagent humanagent requested a review from jhaaaa as a code owner December 1, 2023 14:51
@humanagent humanagent requested a review from a team December 1, 2023 14:51
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Dec 1, 2023 •
edited
Loading

⚠️ No Changeset found

Latest commit: 69bef71

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@humanagent humanagent changed the title XMTP Node JS Improvements xmtp-node-js Dec 1, 2023
@humanagent humanagent requested a review from neekolas December 1, 2023 14:52
rygine
rygine reviewed Dec 1, 2023
View reviewed changes
packages/.DS_Store Outdated
Copy link
Contributor

Choose a reason for hiding this comment

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

not sure how this got through since it's present in .gitignore

neekolas
neekolas reviewed Dec 1, 2023
View reviewed changes
package.json Outdated
@@ -58,5 +58,5 @@
"typescript": "~5.2.2",
"vitest": "^0.34.6"
},
"packageManager": "[email protected]-rc.51"
"packageManager": "[email protected]"
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh ya! There's a stable 4.0 release now

packages/bot-kit-pro/README.md Outdated

### Installation

To install Bot Kit Pro, you can use the yarn package manager:
Copy link
Contributor

Choose a reason for hiding this comment

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

We don't make any requirements that you use Yarn to consume Bot Kit Pro. Just needed to publish new versions

packages/bot-examples/README.md
@@ -1,3 +1,299 @@
# Bot Examples

TODO
This repository contains examples of bots built using the `@xmtp/bot-kit-pro` package. Each bot is designed to demonstrate a different feature or use case.
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for adding this!

@humanagent humanagent removed the request for review from jhaaaa December 4, 2023 20:44
@humanagent humanagent force-pushed the xmtp-node-js branch 2 times, most recently from 65b6fdc to 8a9f16e Compare December 4, 2023 22:38
@humanagent humanagent force-pushed the xmtp-node-js branch 2 times, most recently from 12f0d2d to 69bef71 Compare December 4, 2023 23:23
@humanagent humanagent closed this Dec 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@rygine rygine rygine left review comments

@neekolas neekolas neekolas left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@humanagent @neekolas @rygine