bcgov / SIMS

Student Information Management System. Post-Secondary Student Financial Aid System
Apache License 2.0
24 stars 13 forks source link

Additional transportation calculation updates #3505

Closed Joshua-Lakusta closed 1 month ago

Joshua-Lakusta commented 1 month ago

As a student I want the additional transportation to calculate correctly based on the changes to the forms on #3351.

Acceptance Criteria

Question Response Old Variable New Variable
1. Do you want addtl transport? Yes/No studentDataEligibleForAnAdditionalTransportationAllowance studentDataAdditionallTransportRequested
2. Are you the listed driver? Yes/No n/a studentDataAddtionalTransportListedDriver
3. Are you the owner? Yes/No n/a studentDataAddtionalTransportOwner
4. Weekly KM 1-999 studentDataAdditionalTransportKm no change
5. Is their a clinical practicuum? Yes/No studentDataAdditionalTransportPlacement no change
6. Weekly travel costs 1-999 studentDataAdditionalTransportCost removed
if(offeringDelivered = "onsite")
then 
  offeringWeeks *  dmnPartTimeProgramYearMaximums.limitTransportationAllowance
else
  0

Additional Context Proposed BPM Changes for additional transport (Under AEST-SIMS>Devs>BPM Workflows>#3505,#3351) App & Change Request Questions updated on #3351 New rules for calculation:

Current Flow:

image.png

Test scenario examples

image.png

Technical

andrewsignori-aot commented 1 month ago

@CarlyCotton to add the BPMN for reference. @Joshua-Lakusta to provide test input scenarios and expected outputs.

guru-aot commented 1 month ago

If some report is using the existing additional transportation variables, this changed variable will not be fetched, if a change is required or investigated, please create tickets. @CarlyCotton @Joshua-Lakusta

dheepak-aot commented 1 month ago

As discussed with @Joshua-Lakusta @CarlyCotton there is a minor update to the way how calculatedDataNetWeeklyAdditionalTransportCost is calculated.

image.png image.png