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

Update monetization-related docs #1539

Merged
merged 7 commits into from
Jan 10, 2025
Merged

Conversation

oliwiapolec
Copy link
Collaborator

Deploy preview

Scroll down to the checks section for the link to the deploy preview of your branch.

Links

Link your Jira ticket.

Description

Review and release

Apply changes and resolve conflicts. Once the described functionality lands on prod, let us know on #platform-docs-releases that your PR is ready for release. We will merge and publish the changes.

@oliwiapolec oliwiapolec requested a review from a team as a code owner December 13, 2024 12:20
src/pages/monetization/how-to-monetize-apps/index.mdx Outdated Show resolved Hide resolved
src/pages/monetization/how-to-monetize-apps/index.mdx Outdated Show resolved Hide resolved
src/pages/monetization/how-to-monetize-apps/index.mdx Outdated Show resolved Hide resolved
src/pages/monetization/how-to-monetize-apps/index.mdx Outdated Show resolved Hide resolved
src/pages/monetization/how-to-monetize-apps/index.mdx Outdated Show resolved Hide resolved
src/pages/monetization/billing-api/index.mdx Outdated Show resolved Hide resolved
src/pages/monetization/billing-api/index.mdx Show resolved Hide resolved
```

</CodeSample>

Copy link
Collaborator

Choose a reason for hiding this comment

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

Maybe we should create a short instruction of what to do exactly?

  1. Create a charge in Billing API
  2. Call startTransaction
  3. Display the modal
  4. Register payment status events

Something like that? @SamborGornicz

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

When we confirm this flow is preferred I can write the instruction 😌

Copy link
Collaborator

Choose a reason for hiding this comment

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

Great idea! Here’s the workflow from the developer’s perspective:

  1. Create a Charge: use the Billing API to create a new charge.
  2. Initiate a Transaction: call startTransaction with the registered charge and any metadata. This triggers a modal in the AA (or parent?).
  3. Monitor Events: Set up listeners for transactions related events:
  • webhooks - listen to transaction webhooks
  • agent-app-sdk - listen for transaction events.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The webhooks would be custom here right?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think he meant these webhooks: https://platform.text.com/docs/getting-started/app-guides/app-webhooks
Btw. @SamborGornicz, will there by any new webhooks for this payment method?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes, I was referring to app webhooks. At the moment, I don’t think so. I have some plans regarding payment webhooks data structure, but, it’s unlikely to happen anytime soon.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@ale-kacperczyk would you take a look if this short instruction I wrote sounds ok to you? If yes, I think we're ready to publish this 👀 I just added the imgs as well

@oliwiapolec oliwiapolec merged commit 2c573d9 into master Jan 10, 2025
4 checks passed
@oliwiapolec oliwiapolec deleted the feature/direct-payments-docs branch January 10, 2025 09:31
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.

3 participants