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

Refactor parsing engine to class-based design #3463

Open
5 of 9 tasks
elipe17 opened this issue Feb 10, 2025 · 0 comments · May be fixed by #3470
Open
5 of 9 tasks

Refactor parsing engine to class-based design #3463

elipe17 opened this issue Feb 10, 2025 · 0 comments · May be fixed by #3470
Assignees
Labels
backend dev Refined Ticket has been refined at the backlog refinement

Comments

@elipe17
Copy link

elipe17 commented Feb 10, 2025

Description:
Using the tech memo outlined in #3452 as a guide, refactor the parsing engine from the functional design to the recommended class based design. No work on the FRA parser outlined in the tech memo should be included in this work. Only current parsing code should be refactored.

Acceptance Criteria:
Create a list of functional outcomes that must be achieved to complete this issue

  • Class based parser implemented
  • All current parser tests do not change logically
  • Testing Checklist has been run and all tests pass
  • README is updated, if necessary

Tasks:
Create a list of granular, specific work items that must be completed to deliver the desired outcomes of this issue

  • Update the SchemaManager to handle context for entire file
  • Implement suggested decoder class hierarchy
  • Implement parser class and subclass
  • Update parse logic to leverage new class
  • Run Testing Checklist and confirm all tests pass

Notes:

  • I wanted to see if there was any naming guidance we could use to aggregate TANF/SSP/Tribal with one term since we have FRA as a concise term. I asked Kathryn and this was the response: Just spoke with Alex. Internally they sometimes refer to these files as TDR - TANF Data Report. The implementer can use this info as they see fit.
@lhuxraft lhuxraft changed the title Parser Refactor Refactor parsing engine to class-based design Feb 10, 2025
@lhuxraft lhuxraft added the Refined Ticket has been refined at the backlog refinement label Feb 11, 2025
@elipe17 elipe17 self-assigned this Feb 11, 2025
@elipe17 elipe17 linked a pull request Feb 19, 2025 that will close this issue
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend dev Refined Ticket has been refined at the backlog refinement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants