Update pricing with Private Payment API + refactor with Tabs#67
Open
supermarioblock wants to merge 1 commit intomainfrom
Open
Update pricing with Private Payment API + refactor with Tabs#67supermarioblock wants to merge 1 commit intomainfrom
supermarioblock wants to merge 1 commit intomainfrom
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
Contributor
WalkthroughConverted pricing page from linear layout to tabbed interface with three tabs for Ephemeral Rollup, Private Ephemeral Rollups, and VRF. Reorganized existing content into respective tabs and added new Private Payment API section under the PER tab. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@pages/overview/additional-information/pricing.mdx`:
- Line 37: Fix grammatical errors in the pricing copy: change the phrase
"**Private Payment API** make it simple to send private stablecoin transfers on
Solana Mainnet and/or on an a Private ER." to use correct subject-verb agreement
and articles (e.g., "makes it simple to send private stablecoin transfers on
Solana Mainnet and/or on a Private ER."), and apply similar fixes to the related
sentences at the other instances (the phrases on lines 46 and 51) so they use
correct singular/plural and remove duplicated articles.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro
Run ID: a31d80c3-294e-47da-8e40-808d00e59999
📒 Files selected for processing (1)
pages/overview/additional-information/pricing.mdx
| <Tab title="Private Ephemeral Rollups (PER)"> | ||
|
|
||
| ### Private Payment API | ||
| **Private Payment API** make it simple to send private stablecoin transfers on Solana Mainnet and/or on an a Private ER. |
Contributor
There was a problem hiding this comment.
Fix user-facing grammar in pricing copy (Line 37, Line 46, Line 51).
These lines contain small grammar issues that reduce clarity in public docs.
✍️ Proposed copy edits
- **Private Payment API** make it simple to send private stablecoin transfers on Solana Mainnet and/or on an a Private ER.
+ **Private Payment API** makes it simple to send private stablecoin transfers on Solana Mainnet and/or on a Private ER.
- **PER** support any type of custom computation specified by your smart contract. The pricing fee for PER for custom logic execution is the same as standard ER.
+ **PER** supports any type of custom computation specified by your smart contract. Pricing for PER custom logic execution is the same as standard ER.
- > ⚠️ Note: Costs do not include the transaction to request randomness. On ER transactions are free, on Solana transactions may vary based on your priority fees.
+ > ⚠️ Note: Costs do not include the transaction used to request randomness. On ER, transactions are free; on Solana, transaction costs vary based on priority fees.Also applies to: 46-46, 51-51
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@pages/overview/additional-information/pricing.mdx` at line 37, Fix
grammatical errors in the pricing copy: change the phrase "**Private Payment
API** make it simple to send private stablecoin transfers on Solana Mainnet
and/or on an a Private ER." to use correct subject-verb agreement and articles
(e.g., "makes it simple to send private stablecoin transfers on Solana Mainnet
and/or on a Private ER."), and apply similar fixes to the related sentences at
the other instances (the phrases on lines 46 and 51) so they use correct
singular/plural and remove duplicated articles.
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary by CodeRabbit