Closed
Conversation
- Complete Jekyll configuration (_config.yml, Gemfile, .gitignore) - All section index pages with proper navigation - Complete backend documentation copied line-by-line: - Phaxio setup (PHAXIO_SETUP.md) - Sinch setup (SINCH_SETUP.md) - SIP/Asterisk setup (SIP_SETUP.md) - Images & PDFs guide (IMAGES_AND_PDFS.md) - Contributing guide with complete content - GitHub Actions workflow for automated deployment - Main index page from README with updated links Still copying remaining docs: MCP, API, Security, SDKs, etc. All original docs remain untouched in their original locations.
MAJOR MILESTONE: All core documentation copied with complete fidelity ✅ COMPLETED - Every single line copied from original docs: - MCP Integration (220 lines) - Complete AI integration guide - API Reference (80 lines) - Complete REST API documentation - HIPAA Requirements (162 lines) - Complete security compliance guide - OAuth Setup (89 lines) - Complete identity provider configuration - Client SDKs (65 lines) - Complete Node.js and Python SDK docs - Troubleshooting (112 lines) - Complete problem resolution guide - Phaxio E2E Test (72 lines) - Complete end-to-end testing guide ✅ STRUCTURE COMPLETE: - All Jekyll configuration files (_config.yml, Gemfile, .gitignore) - All section index pages with proper navigation hierarchy - GitHub Actions workflow for automated deployment - Complete asset structure with demo video - Proper Jekyll frontmatter for all documents ✅ NAVIGATION STRUCTURE: - Getting Started (Contributing guide) - Backends (Phaxio, Sinch, SIP/Asterisk, Images & PDFs) - AI Integration (MCP Integration) - Security (HIPAA Requirements, OAuth Setup) - Development (API Reference, SDKs, Troubleshooting, E2E Test) REMAINING: Node MCP README, SDK READMEs, API Changelog All original docs remain completely untouched in original locations.
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
- Updated iOS app documentation - Updated multi-repo architecture info - Updated branch policies for multi-repo setup Auto-synced by sync-agents-md.sh
…ild/sign instructions
DMontgomery40
temporarily deployed
to
github-pages
…/Node/Python snippets; document refresh semantics and inbound examples
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
…ces and add asterisk profile
…end support for sending and receiving faxes in v3
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
- Add hybrid configuration troubleshooting section - Update environment variable references to include FAX_OUTBOUND_BACKEND/FAX_INBOUND_BACKEND - Update deployment docs for dynamic backend service requirements - Add hybrid backend mention to main index features - Add Hybrid Setups to table of contents 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
had a problem deploying
to
github-pages
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
DMontgomery40
temporarily deployed
to
github-pages
DMontgomery40
force-pushed
the
docs-jekyll-site
branch
from
4aa9712 to
4bbb6be
Compare
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
DMontgomery40
temporarily deployed
to
github-pages
GitHub Actions
Inactive
| "faxNumber": to, | ||
| } | ||
| resp = await client.post(url, headers=self._headers(), data=data, files=files) | ||
| if resp.status_code >= 400: |
There was a problem hiding this comment.
Bug: File Handle Leak in Fax Sending Method
The send_fax_file method has a file handle leak. The file opened with open(file_path, "rb") is not explicitly closed, which can lead to file descriptor exhaustion, especially under high load or if exceptions occur.
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.
No description provided.