Add comprehensive onboarding documentation and configuration management#1
Open
tqvdang wants to merge 5 commits into
Open
Add comprehensive onboarding documentation and configuration management#1tqvdang wants to merge 5 commits into
tqvdang wants to merge 5 commits into
Conversation
- Add dual session storage (provider:payment_id + fallback keys) - Implement automatic session restoration in elicitation flow - Handle client timeouts during PayPal approval process - Add connection timeout configuration (10 minutes) - Update README with session restoration documentation - Add providers.json for runtime configuration switching - Remove unnecessary debug/session fix files
- Add list_sessions_by_provider method to session storage interface - Implement active payment discovery in _check_for_completed_payments - Check all pending payments on tool invocation regardless of session ID - Auto-execute tools when payments completed externally - Solves reconnection issue where new session IDs prevent recovery
- Add detailed README with 5-step Quick Start guide - Add mock provider support for testing without API keys - Add .env.example template for environment variables - Add configuration loading with environment variable substitution - Add multiple payment flow support (elicitation, two_step, progress, list_change) - Add uv run mcp install instructions for Claude Desktop setup - Add MCP Inspector testing guide - Add troubleshooting section with common issues - Add PayMCP library update instructions (local vs published) - Add support for multiple payment providers (Mock, Stripe, PayPal, Walleot, Square) - Add get_config and switch_payment_config tools - Update gitignore patterns for Python - Remove test_session_restoration.py (moved to paymcp-flow-tester)
Changes: - Update providers.json: list_change -> dynamic_tools - Update server.py PaymentFlow enum references - Update README.md documentation with new flow name Aligns with paymcp library rename of LIST_CHANGE to DYNAMIC_TOOLS flow. All functionality remains the same, only naming has changed for clarity.
tqvdang
force-pushed
the
feat/eng-174-implement-list-change
branch
from
ce729a8 to
4749c64
Compare
- Add resubmit to flow_map with PaymentFlow.RESUBMIT - Add resubmit to availableFlows in providers.json
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
Adds comprehensive developer onboarding documentation and runtime configuration management to improve the demo server's usability and accessibility.
Changes
Documentation
uv run mcp installinstructions for Claude Desktop setup (recommended)Configuration Management
.env.exampletemplate for environment variablesproviders.jsonget_configtool to inspect current configurationswitch_payment_configtool for runtime provider/flow switchingPayment Features
Code Cleanup
test_session_restoration.py(moved to centralized paymcp-flow-tester)Testing
Tested with:
uv run mcp install- Claude Desktop integrationBreaking Changes
None - fully backward compatible.