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

[FYST-1727] special program az #5473

Merged
merged 5 commits into from
Jan 30, 2025
Merged

[FYST-1727] special program az #5473

merged 5 commits into from
Jan 30, 2025

Conversation

jnf
Copy link
Contributor

@jnf jnf commented Jan 29, 2025

Link to pivotal/JIRA issue

FYST-1727

Is PM acceptance required? (delete one)

Yes - don't merge until JIRA issue is accepted!

What was done?

  • Adds a new element to the return header in AZ xml submissions.
  • SpecialProgram is an optional node that has remain unused; AZ folks asked if we could start including it to help them in their record keeping/stat building.

How to test?

  • Make an AZ return; it should have <SpecialProgram>Direct File</SpecialProgram> right after <ReturnType> in the <ReturnHeaderState> block.
  • No other state return should include <SpecialProgram>

@jnf jnf added the wip denotes a work in progress that isn't ready for formal review label Jan 29, 2025
Copy link

Heroku app: https://gyr-review-app-5473-b680c13da290.herokuapp.com/
View logs: heroku logs --app gyr-review-app-5473 (optionally add --tail)

@jnf jnf marked this pull request as ready for review January 29, 2025 17:11
@jnf jnf removed the wip denotes a work in progress that isn't ready for formal review label Jan 29, 2025
@@ -185,6 +187,10 @@ def schema_file
SchemaFileLoader.load_file("us_states", "unpacked", "AZIndividual2024v2.0", "AZIndividual", "IndividualReturnAZ140.xsd")
end

def special_program
"<SpecialProgram>Direct File</SpecialProgram>"
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 dunno if there's a fancy way to do this, but this works.

Copy link
Member

Choose a reason for hiding this comment

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

I think it would be something kinda ridiculous like

Nokogiri::XML::Builder.new(encoding: 'UTF-8').send("SpecialProgram", "Direct File")

what you have is probably more clear

@@ -185,6 +187,10 @@ def schema_file
SchemaFileLoader.load_file("us_states", "unpacked", "AZIndividual2024v2.0", "AZIndividual", "IndividualReturnAZ140.xsd")
end

def special_program
"<SpecialProgram>Direct File</SpecialProgram>"
Copy link
Member

Choose a reason for hiding this comment

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

I think it would be something kinda ridiculous like

Nokogiri::XML::Builder.new(encoding: 'UTF-8').send("SpecialProgram", "Direct File")

what you have is probably more clear

@jnf jnf merged commit 5748041 into main Jan 30, 2025
7 checks passed
@jnf jnf deleted the jnf/fyst-1727/special-program-az branch January 30, 2025 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants