Skip to content

Conversation

@embarnard
Copy link
Contributor

@embarnard embarnard commented Nov 25, 2024

Link to pivotal/JIRA issue

https://codeforamerica.atlassian.net/browse/FYST-1193?atlOrigin=eyJpIjoiODYzMzAzN2ViODEzNDNiNWE3YWM4ZGMzYTBkMTM0YjIiLCJwIjoiaiJ9

Is PM acceptance required? (delete one)

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

Reminder: merge main into this branch and get green tests before merging to main

What was done?

  • We now redirect away from the data review page, moved efile device id collection to the income review page
  • Initially we tried changing the hidden button on the waiting to load page to submit the device id but this proved risky so opted for taking another approach
  • The data review page still exists but only for non-prod environments since we needed a page that would hold that edit DF XML button in testing environments

How to test?

  • Test that AZ, ID, MD, NC and NJ all skip the data review page and that we still have have these fields filled out in the xml auth header:
xml.TransmissionDetail do
          xml.InitialCreation do
            xml.IPAddress do
              xml.IPv4AddressTxt 
              xml.IPv6AddressTxt 
            end
            xml.IPTs 
            xml.DeviceId 
            xml.DeviceTypeCd 
          end
end

@github-actions
Copy link

Heroku app: https://gyr-review-app-5029-0fb8eede6440.herokuapp.com/
View logs: heroku logs --app gyr-review-app-5029 (optionally add --tail)

income_and_deductions_title: Income and deductions
partially_complete_description: Partially complete
state_tax_withheld_title: "%{name} state tax withheld"
title: Your federal tax return is now transferred. We completed some sections to save you time.
Copy link
Contributor

@anisharamnani anisharamnani Nov 26, 2024

Choose a reason for hiding this comment

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

[dust] 🤔 since this page is now only going to be visible to those on non-prod environments, it might be helpful to see that in the label. For example "This page is only visible on development" The current description is more filer saving.

click_on "Continue"

expect(page).to have_text "Your federal tax return is now transferred."
end
Copy link
Contributor

Choose a reason for hiding this comment

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

[pebble] removing these expectations leaves the test in a weird state. the last line in each of these tests is clicking on "Continue." I think it would be preferable to have an expectation.

Is there an expectation that can be added here after clicking on continue? Possibly testing that the SSN is being added to the DB?

Copy link
Contributor

@anisharamnani anisharamnani left a comment

Choose a reason for hiding this comment

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

i've got two comments. one pebble & dust. otherwise this looks good! thanks @embarnard!

embarnard and others added 6 commits November 26, 2024 14:58
Co-authored-by: Anisha Ramnani <[email protected]>
Co-authored-by: Anisha Ramnani <[email protected]>
Co-authored-by: Anisha Ramnani <[email protected]>
Co-authored-by: Anisha Ramnani <[email protected]>
Co-authored-by: Anisha Ramnani <[email protected]>
@embarnard embarnard merged commit a4f00eb into main Dec 2, 2024
4 checks passed
@embarnard embarnard deleted the FYST-1193-hide-data-review branch December 2, 2024 19:02
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.

3 participants