-
Notifications
You must be signed in to change notification settings - Fork 35
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
INT B-22272 #14752
INT B-22272 #14752
Conversation
…cs, add tests, and refine refine refine
B 20636 MAIN - #26 Dependency Scan (Severity High) - ip 2.0.1 (DEV)
migrations/app/schema/20250123173216_add_destination_queue_db_func_and_gbloc_view.up.sql
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reapproving
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seeing this error here when clicking on a move in the destination queue and then hitting the back button in the chrome browser. I do not see the same behavior for the normal queue
To recreate:
- View the destination queue
- Click on a move there to see the Move Details
- Click the back button on the browser
Server logs:
2025-02-07T17:55:24.138Z INFO middleware/request_logger.go:87 Request {"git_branch": "B-21322-MAIN", "git_commit": "2ae6dc82f22fb8359aa5561d49a1bf628ca89777", "host": "officelocal:3000", "milmove_trace_id": "cb785013-2ab4-4b99-8131-0e8b8bbd9b7a", "session_id": "9AgkB61p1tPYFnQhnr_ENRbYbQqT2JToyNHh_LSzUgU", "accepted-language": "", "content-length": 0, "host": "officelocal:3000", "method": "GET", "named-server": "no-tls", "protocol-version": "HTTP/1.1", "referer": "http://officelocal:3000/destination-requests", "source": "127.0.0.1:64958", "url": "/internal/users/logged_in", "user-agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/131.0.0.0 Safari/537.36", "protocol": "http", "x-csrf-token": "R1cKgOl623GAkochdDcHZflJxugDNz/tRbhf9f7k/31J+X3lwaD2iJG/ssuS/SxAynJVml86cdUJPpaIt8f5RQ==", "headers": 8, "user-id": "8aa60d81-a83a-4990-8537-f44ae407b5a2", "office-user-id": "733a2366-e965-422d-b811-bfa6fd21aca4", "duration": "16.445041ms", "resp-size-bytes": 3686, "resp-status": 200}
[POP] 2025/02/07 17:55:24 sql - SELECT office_users.active, office_users.created_at, office_users.edipi, office_users.email, office_users.first_name, office_users.id, office_users.last_name, office_users.middle_initials, office_users.other_unique_id, office_users.rejection_reason, office_users.status, office_users.telephone, office_users.transportation_office_id, office_users.updated_at, office_users.user_id FROM office_users AS office_users WHERE office_users.id = $1 LIMIT 1 | ["733a2366-e965-422d-b811-bfa6fd21aca4"] (conn=tx-995762270905164925-SqKatM, tx=995762270905164925)
[POP] 2025/02/07 17:55:24 sql - SELECT transportation_offices.address_id, transportation_offices.created_at, transportation_offices.gbloc, transportation_offices.hours, transportation_offices.id, transportation_offices.latitude, transportation_offices.longitude, transportation_offices.name, transportation_offices.note, transportation_offices.provides_ppm_closeout, transportation_offices.services, transportation_offices.shipping_office_id, transportation_offices.updated_at FROM transportation_offices AS transportation_offices WHERE id = $1 | [cf5fee51-2f62-4de2-adee-6b48134fd597] (conn=tx-995762270905164925-SqKatM, tx=995762270905164925)
[POP] 2025/02/07 17:55:24 sql - SELECT privileges.created_at, privileges.id, privileges.privilege_name, privileges.privilege_type, privileges.updated_at FROM privileges AS privileges JOIN users_privileges ON users_privileges.privilege_id = privileges.id WHERE users_privileges.deleted_at IS NULL AND users_privileges.user_id = $1 | [8aa60d81-a83a-4990-8537-f44ae407b5a2] (conn=tx-995762270905164925, tx=995762270905164925)
[POP] 2025/02/07 17:55:24 sql - UPDATE "moves" AS moves SET "additional_documents_id" = :additional_documents_id, "approvals_requested_at" = :approvals_requested_at, "approved_at" = :approved_at, "available_to_prime_at" = :available_to_prime_at, "billable_weights_reviewed_at" = :billable_weights_reviewed_at, "cancel_reason" = :cancel_reason, "closeout_office_id" = :closeout_office_id, "contractor_id" = :contractor_id, "counseling_transportation_office_id" = :counseling_transportation_office_id, "excess_unaccompanied_baggage_weight_acknowledged_at" = :excess_unaccompanied_baggage_weight_acknowledged_at, "excess_unaccompanied_baggage_weight_qualified_at" = :excess_unaccompanied_baggage_weight_qualified_at, "excess_weight_acknowledged_at" = :excess_weight_acknowledged_at, "excess_weight_qualified_at" = :excess_weight_qualified_at, "excess_weight_upload_id" = :excess_weight_upload_id, "financial_review_flag" = :financial_review_flag, "financial_review_flag_set_at" = :financial_review_flag_set_at, "financial_review_remarks" = :financial_review_remarks, "locator" = :locator, "lock_expires_at" = :lock_expires_at, "locked_by" = :locked_by, "orders_id" = :orders_id, "ppm_type" = :ppm_type, "prime_counseling_completed_at" = :prime_counseling_completed_at, "reference_id" = :reference_id, "sc_assigned_id" = :sc_assigned_id, "service_counseling_completed_at" = :service_counseling_completed_at, "shipment_seq_num" = :shipment_seq_num, "show" = :show, "status" = :status, "submitted_at" = :submitted_at, "tio_assigned_id" = :tio_assigned_id, "tio_remarks" = :tio_remarks, "too_assigned_id" = :too_assigned_id, "updated_at" = :updated_at WHERE moves.id = :id | ["1ed3a1d5-abe2-419e-82f4-a5baa30f5b5d"] (conn=tx-995762270905164925, tx=995762270905164925)
[POP] 2025/02/07 17:55:24 sql - ROLLBACK Transaction --- (conn=tx-995762270905164925, tx=995762270905164925)
2025-02-07T17:55:24.145Z INFO middleware/request_logger.go:87 Request {"git_branch": "B-21322-MAIN", "git_commit": "2ae6dc82f22fb8359aa5561d49a1bf628ca89777", "host": "officelocal:3000", "milmove_trace_id": "8e294ac6-6ef7-4a48-8ecc-d9302f917a0a", "session_id": "9AgkB61p1tPYFnQhnr_ENRbYbQqT2JToyNHh_LSzUgU", "accepted-language": "", "content-length": 0, "host": "officelocal:3000", "method": "GET", "named-server": "no-tls", "protocol-version": "HTTP/1.1", "referer": "http://officelocal:3000/destination-requests", "source": "127.0.0.1:64956", "url": "/ghc/v1/queues/destination-requests?page=1&perPage=20&sort=status&order=asc", "user-agent": "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/131.0.0.0 Safari/537.36", "protocol": "http", "x-csrf-token": "R1cKgOl623GAkochdDcHZflJxugDNz/tRbhf9f7k/31J+X3lwaD2iJG/ssuS/SxAynJVml86cdUJPpaIt8f5RQ==", "headers": 8, "user-id": "8aa60d81-a83a-4990-8537-f44ae407b5a2", "office-user-id": "733a2366-e965-422d-b811-bfa6fd21aca4", "duration": "23.213167ms", "resp-size-bytes": 0, "resp-status": 500}
pments.pro_gear_weight
06655b8
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alright ran through quite a few moves and figuring out the expected GBLOCs to check for them was fun. Looks good!
Wonder if we should start considering adding test scenarios to the test harness- to assist with folks setting up data to test PRs |
Absolutely - had the same thought |
Not a bad idea.. Sounds like something I can knock out during Sprint 5 after breakdowns. I'd be down for it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
reviewed frontend specific.. looks good
Agility ticket
Summary
This has been previously tested using our typical queue backend code, but this new BL is using a db function instead with an addition of a
move_to_dest_gbloc
view, which will handle all of the unique affiliation filters we need.This PR has quite a bit of setup so prepare yer butts. Unfortunately the only way to test this thoroughly is to create a metric butt ton of moves.
Per the BL in Agility, we want to add a new "Destination Requests" queue for TOO. This queue will contain moves that are in
Approvals Requested
status and have the following criteria based on the SHIPMENT'S DESTINATION ADDRESS GBLOC, not the destination duty location on the orders:SUBMITTED
status)REQUESTED
status)APPROVALS REQUESTED
statusThere are these exceptions for
USMC
customers (service_members.affiliation
)USMC
moves go into theUSMC
GBLOCThere are these exceptions for
USAF
andSpace Force
customers (service_members.affiliation
)re_rate_areas.name
) it needs to go to theMBFL
GBLOCre_rate_areas.name
), it needs to go to the destination address' respective GBLOCJEAT
GBLOCI would highly advise you to set up a tracker doc (I just used the Mac Notes app) so you can mark which GBLOC the moves should be in, as well as move codes, etc. - here's what I did.
How to test USMC
Destination Requests
tabJEAT
is for AK Zone II &MAPK
is for AK Zone IVHow to test USAF/SF in Alaska Zone II & non-Alaska Zone II
Destination Requests
tabMBFL
TOO, you should see AT A MINIMUM, the Alaska Zone II moves (1 for USAF, 1 for SF)How to test remaining queues
JEAT
GBLOC)JEAT
TOO and view theDestination Requests
tabAdditional testing
Screenshots