Skip to content

feat: Add Letter feature integration with testing and documentation#86

Open
gariwat wants to merge 2 commits into
nowtec:mainfrom
gariwat:feature/letter-integration
Open

feat: Add Letter feature integration with testing and documentation#86
gariwat wants to merge 2 commits into
nowtec:mainfrom
gariwat:feature/letter-integration

Conversation

@gariwat

@gariwat gariwat commented Dec 19, 2025

Copy link
Copy Markdown
  • Add html-pdf dependency to composer package.json
  • Add mammoth and pdf-parse dependencies to strapi package.json
  • Create database migration script for letter feature
  • Add integration test suite for letter channel
  • Create complete integration documentation
  • Add deployment checklist

Pull Request

Thank you for contributing to nowCRM!
Please complete this template before submitting your PR.


Summary


Related Issues / Tickets


Type of Change

  • feat – New feature
  • fix – Bug fix
  • chore – Config change or technical chore
  • docs – Documentation update

Description


How to Test


Checklist

Before submitting, make sure you have:

  • Run pnpm lint:fix && pnpm test without errors
  • Updated or added documentation if needed
  • Linked related issues or tasks
  • Verified functionality locally

Screenshots / Logs (if applicable)


Notes for Reviewers

gariwat and others added 2 commits December 19, 2025 15:31
- Add html-pdf dependency to composer package.json
- Add mammoth and pdf-parse dependencies to strapi package.json
- Create database migration script for letter feature
- Add integration test suite for letter channel
- Create complete integration documentation
- Add deployment checklist

Co-authored-by: openhands <openhands@all-hands.dev>
- Implement StrapiLetterClient for Letter-Template API integration
- Add sendLetter function with PDF generation and email attachment support
- Create LetterEmailIntegration for complete delivery workflow
- Add comprehensive test suite for letter channel functionality
- Integrate with existing email channel for PDF delivery
- Remove console.log statements from production code
- Add environment configuration for Strapi API

Co-authored-by: openhands <openhands@all-hands.dev>
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.

1 participant