raft-tech / TANF-app

Repo for development of a new TANF Data Reporting System
Other
16 stars 3 forks source link

Update TANF-app/doc/Background/ #1521

Closed valcollignon closed 2 years ago

valcollignon commented 2 years ago

Context

After #1515 is merged, TANF-app/doc/Background/ subdirectory markdowns need to be reviewed and updated.

ACs:

Tasks:

Notes:

Part of the #1477 documentation update epic

valcollignon commented 2 years ago

Here is the README.md draft for TANF-app/doc/Background.

ADPennington commented 2 years ago

@valcollignon @lfrohlich hackmd

Here is the README.md draft for TANF-app/doc/Background.

@valcollignon @lfrohlich this hackmd is ready for review.

valcollignon commented 2 years ago

@valcollignon @lfrohlich this hackmd is ready for review.

LGTM

lfrohlich commented 2 years ago

I made a few edits and responded to comments. Looks good to me! @ADPennington

ADPennington commented 2 years ago

I made a few edits and responded to comments. Looks good to me! @ADPennington

will address this change in local branch.

ADPennington commented 2 years ago

@valcollignon @sreedevip the tdrs hackmd is ready for your review and input. Please note:

Please ping me when the hackmds have been finalized, so I can open the PR with changes to this subdir.

sreedevip commented 2 years ago

@ADPennington I love the level of detail on this document. I've left a couple of comments about

ADPennington commented 2 years ago

@sreedevip --

valcollignon commented 2 years ago

All links have been updated/created. @sreedevip -- I've added a few comments for your review/input

sreedevip commented 2 years ago

@valcollignon I've incorporated the GIthub link as requested, but also added a caveat to that in the comments section of hackmd

valcollignon commented 2 years ago

@ADPennington this should be good to go, unless you want to wait to merge until the subdir organization changes are complete in #1527, per @sreedevip's comment above.

ADPennington commented 2 years ago

@ADPennington this should be good to go, unless you want to wait to merge until the subdir organization changes are complete in #1527, per @sreedevip's comment above.

@valcollignon i think this will be ready once the figma diagram is ready. also re: @sreedevip 's q about waiting to merge this until after #1527 is addressed, i'd like to get this one merged before the end of the week, so please let me know if we think 1527 will be ready before then. If not, we can create a clean-up ticket to fix the links.

valcollignon commented 2 years ago

Added task for @sreedevip to add Figma link to Current-TDRS.md.

@sreedevip - do you think #1527 can be completed by the end of this week?

sreedevip commented 2 years ago

Would this be better off in the Background folder? https://github.com/raft-tech/TANF-app/blob/raft-tdp-main/docs/User-Experience/Flat%20File%20Metadata%20Guide.md @ADPennington @valcollignon @reitermb cc @lfrohlich

ADPennington commented 2 years ago

Would this be better off in the Background folder? https://github.com/raft-tech/TANF-app/blob/raft-tdp-main/docs/User-Experience/Flat%20File%20Metadata%20Guide.md @ADPennington @valcollignon @reitermb cc @lfrohlich

since its based on research raft did, i don't think it needs to be moved @sreedevip

ADPennington commented 2 years ago

figma diagram embedded in tdrs hackmd.

sreedevip commented 2 years ago

Based on @ADPennington's comment above, does this need to be somewhere else? https://github.com/raft-tech/TANF-app/blob/raft-tdp-main/docs/User-Experience/STT%20attribute%20guide.md It doesn't seem to be a direct impact for research done by Raft. Could you confirm @reitermb ? cc @valcollignon