Closed raftmsohani closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 93.10%. Comparing base (
00ac3d1
) to head (b2ba38e
). Report is 3 commits behind head on develop.
Tested on my phone (my phone had an ip address in the new ip range) and I could access. Tried prod, and I could not access.
@ADPennington : Raft has the changes, if you have T-mobile then you should be able to test this. I did the test and it worked.
Tested on my phone (my phone had an ip address in the new ip range) and I could access. Tried prod, and I could not access.
thank you! unfortunately we do not have the tools to test this. could you include the relevant logs in this PR @raftmsohani ?
Tested on my phone (my phone had an ip address in the new ip range) and I could access. Tried prod, and I could not access.
thank you! unfortunately we do not have the tools to test this. could you include the relevant logs in this PR @raftmsohani ?
Here, first we try to login to qasp, which doesn't have the whitelisted IP:
As we can see, the ip address is blocked, now let's try RAFT which has the change:
Summary of Changes
Provide a brief summary of changes Pull request closes #3023 _
How to Test
List the steps to test the PR These steps are generic, please adjust as necessary.
Deliverables
More details on how deliverables herein are assessed included here.
Deliverable 1: Accepted Features
Checklist of ACs:
lfrohlich
and/oradpennington
confirmed that ACs are met.Deliverable 2: Tested Code
CodeCov Report
comment in PR)CodeCov Report
comment in PR)Deliverable 3: Properly Styled Code
Deliverable 4: Accessible
iamjolly
andttran-hub
using Accessibility Insights reveal any errors introduced in this PR?Deliverable 5: Deployed
Deliverable 6: Documented
Deliverable 7: Secure
Deliverable 8: User Research
Research product(s) clearly articulate(s):