jrh3k5 / freegames-xmtp Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 852 KB

An XMTP bot that broadcasts announcements of free games over XMTP to subscribers

License: Apache License 2.0

JavaScript 99.11% Shell 0.89%

freegames-xmtp's People

Contributors

Watchers

freegames-xmtp's Issues

Only subscribe on ETH send if in the allowlist

Number of attributes exceeded

We're no longer able to enqueue game data due to this error:

InvalidParameterValue: Number of message attributes [11] exceeds the allowed maximum [10]

Missing trailing zero in price

This should likely be "originally $2.50".

Update 'subscribe' command to tell users to send ETH to the subscription address

Add ability to close registrations

Expire subscriptions after a certain date

Add automatic cleanup of subscriptions to handle free trials.

Handle prime store better

The prime store value isn't getting handled well:

This should be "Amazon Prime", not just "prime".

Consider, also, adding a disclaimer that this is only for Amazon Prime subscribers.

Describe the supported network in messaging

The messages simply say "send ETH" right now, but don't tell the user what network to use.

Undefined original price

Got a notification of a game with an undefined original price - perhaps this game had no original USD price?

Image cache doesn't work

Looks like cached images don't read out correctly - getting this error:

xmtp-webhook-1  | Failed to send image for game ID 565940 TypeError: Method get TypedArray.prototype.length called on incompatible receiver [object Object]
xmtp-webhook-1  |     at get length [as length] (<anonymous>)
xmtp-webhook-1  |     at Object.encode (/home/node/app/node_modules/@xmtp/proto/ts/dist/cjs/message_contents/content.pb.js:152:29)
xmtp-webhook-1  |     at Client.encodeContent (file:///home/node/app/node_modules/@xmtp/xmtp-js/dist/index.js:5412:39)
xmtp-webhook-1  |     at async ConversationV2.send (file:///home/node/app/node_modules/@xmtp/xmtp-js/dist/index.js:1735:25)
xmtp-webhook-1  |     at async Notifier.sendGameImage (file:///home/node/app/xmtp/notify/notifier.js:46:9)
xmtp-webhook-1  |     at async Notifier.notify (file:///home/node/app/xmtp/notify/notifier.js:16:13)
xmtp-webhook-1  |     at async Consumer.handleMessage (file:///home/node/app/queue/user_notification.js:33:9)
xmtp-webhook-1  |     at async Consumer.executeHandler (/home/node/app/node_modules/sqs-consumer/dist/consumer.js:373:26)
xmtp-webhook-1  |     at async Consumer.processMessage (/home/node/app/node_modules/sqs-consumer/dist/consumer.js:254:34)
xmtp-webhook-1  |     at async Promise.all (index 0)

Handle large images more gracefully

Consider implementing remote attachments or, failing that, at least don't try to attach images that are too large and generate this error:

Failed to send image for game ID 111718 GrpcError: message too big
    at GrpcError.fromObject (file:///opt/freegames-xmtp/node_modules/@xmtp/xmtp-js/dist/index.js:2209:16)
    at HttpApiClient._publish (file:///opt/freegames-xmtp/node_modules/@xmtp/xmtp-js/dist/index.js:2300:33)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Client.publishEnvelopes (file:///opt/freegames-xmtp/node_modules/@xmtp/xmtp-js/dist/index.js:5365:9)
    at async ConversationV2.send (file:///opt/freegames-xmtp/node_modules/@xmtp/xmtp-js/dist/index.js:1743:9)
    at async Notifier.sendGameImage (file:///opt/freegames-xmtp/xmtp/notify/notifier.js:46:9)
    at async Notifier.notify (file:///opt/freegames-xmtp/xmtp/notify/notifier.js:16:13)
    at async Consumer.handleMessage (file:///opt/freegames-xmtp/queue/user_notification.js:35:9)
    at async Consumer.executeHandler (/opt/freegames-xmtp/node_modules/sqs-consumer/dist/consumer.js:373:26)
    at async Consumer.processMessage (/opt/freegames-xmtp/node_modules/sqs-consumer/dist/consumer.js:254:34) {
  code: 3
}

I'm attaching the offending image.

Implement periodic subscription expiry management

Implement a process to run periodically to deactivate expired subscriptions.

Send user a 'welcome' message when they send a payment to subscribe

Move game details resolution to game notification queue consumer

If a single game in a webhook request fails, the whole batch fails. This happened when a request was rate limited. Change the architecture to have the webhook enqueue just the game ID and have the consumer make the call to get the game details.

If a user submits ETH before their subscription expires, increment their current expiry as appropriate

Allow one month free trial

Allow for a one-month free trial. Have the expiration messaging flex on whether or not the expiration is for a free trial.

Add concept of 'free forever' for default notification accounts

To ensure that the default notification accounts - which can be used for troubleshooting and monitoring - never have to pay ETH to subscribe, add a 'free forever' concept to the subscriptions model.

Exclude DLC

Somehow, a DLC listing was sent out.

jrh3k5 / freegames-xmtp Goto Github PK

freegames-xmtp's People

Contributors

Watchers

freegames-xmtp's Issues

Recommend Projects

Recommend Topics

Recommend Org

Jobs