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

Create data-flow-sequence-diagram.md #73133

Merged
merged 6 commits into from
Feb 14, 2024
Merged

Conversation

hinzed1127
Copy link
Contributor

No description provided.

@hinzed1127 hinzed1127 requested a review from kjduensing January 8, 2024 19:17
@hinzed1127 hinzed1127 mentioned this pull request Jan 8, 2024
4 tasks
@@ -0,0 +1,33 @@
The basic sequence for most BTSSS is straightforward:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice high-level diagram! I'm wondering to what level we should address these points on the Readiness Review ticket:

  • Data Flow Diagram:
    • This diagram must illustrate the following specifics.
    • What data is collected or used, and where, including information such as credentials used by this system?
    • Where is the data is stored and how, including information such as any encryption used?
    • How is the data transferred, including information such as any encryption used?
    • Who accesses the data and in what capacity (read or read-write)?
    • What is the audit trail of data access and manipulation?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm leaning toward answering these in-diagram from our (VA.gov) perspective and not try to answer for the backing API team.

What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think that's fine, whether that's in the diagram or any additional information below the diagram

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's add it in the diagram. Let me know if you need help fitting something into diagram form

@hinzed1127
Copy link
Contributor Author

@kjduensing I cleaned up the non-diagram stuff to add in things discussed above, i.e., a few outstanding questions related to certain steps in the diagram, and an appendix linking to the Sign-in Service auth sequence diagram. Considering that sufficient for now.

@hinzed1127 hinzed1127 merged commit bde5d8e into master Feb 14, 2024
3 checks passed
@hinzed1127 hinzed1127 deleted the btsss-sequence-diagram branch February 14, 2024 17:19
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.

2 participants