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 document delivery services #11293

Closed
thorwolpert opened this issue Feb 24, 2022 · 3 comments
Closed

Update document delivery services #11293

thorwolpert opened this issue Feb 24, 2022 · 3 comments
Labels

Comments

@thorwolpert
Copy link
Collaborator

thorwolpert commented Feb 24, 2022

Description

The document delivery service for print jobs needs to be changed to be batch job style instead of even style.

The following sequence outlines the suggested solution.

sequenceDiagram
PPR API->>PubSub: Send msg that doc should be generated
loop GenerateDocument
    PubSub->>DocumentGeneration: CreateDocument
    DocumentGeneration->>Cloud Secrets: Get DailStorage Folder Name
    Note right of Cloud Secrets: Folder Format CCYYMMDD
    DocumentGeneration->>DeadLetterSub: Error in Creating Documents
    DocumentGeneration->>DailyStorage: Save Documents
end
Note right of Cloud Scheduler: Agreed Delivery Time: 21:00
Cloud Scheduler->>Storage: Create new DailyStorage
Cloud Scheduler->>Cloud Secrets: Update to new DailyStorage
Cloud Scheduler->>DocumentDelivery: Start Job
DocumentDelivery->>DailyStorage: Create Zip Folder & Audit File
DocumentDelivery->>BCMail: sftp zip & txt
Loading
@kialj876
Copy link
Collaborator

kialj876 commented Feb 24, 2022

This looks good to me! Nice diagram 👍

Where you have the cloud scheduler 'update to new dailyStorage' in client secrets -- thats the same 'dailyStorage folder name' that the document generation uses correct?

And I guess if the sftp fails after a certain amount of retries we just save the zip/txt to a backup db or something?

@doug-lovett
Copy link
Collaborator

Nice diagram. Very helpful.

@thorwolpert
Copy link
Collaborator Author

Good point, I wasn't going to save the zip/txt as it is easily recreated and to save storage costs.
But we should figure out how to notify people that user manual action is required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants