Skip to content

Don't disable temporary vehicle immediately when ending permit#567

Merged
mhieta merged 2 commits intodevelopfrom
dont-disable-temporary-vehicle-immediatly
Mar 6, 2025
Merged

Don't disable temporary vehicle immediately when ending permit#567
mhieta merged 2 commits intodevelopfrom
dont-disable-temporary-vehicle-immediatly

Conversation

@tonipel
Copy link
Contributor

@tonipel tonipel commented Jan 24, 2025

Description

Don't disable temporary vehicle immediatly when ending permit.

Context

Previously temporary vehicles were disabled immediatly always when used ended their permit.

How Has This Been Tested?

Manually and automated testing.

Manual Testing Instructions for Reviewers

If user has a temporary vehicle and ends their permit, the temporary vehicle is disabled only when the permit ends, or as before if permit has a longer period left than temporary vehicle.

Screenshots

@tonipel tonipel requested a review from mhieta January 24, 2025 13:00
@tonipel tonipel force-pushed the dont-disable-temporary-vehicle-immediatly branch from 7bb65d4 to f0e49b4 Compare January 24, 2025 13:03
@mhieta mhieta changed the title Don't disable temporary vehicle immediatly when ending permit Don't disable temporary vehicle immediately when ending permit Jan 27, 2025
Copy link
Contributor

@mhieta mhieta left a comment

Choose a reason for hiding this comment

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

Added few improvement ideas.
Also testsuite seems to fail, so needs to be fixed.

@tonipel tonipel force-pushed the dont-disable-temporary-vehicle-immediatly branch from f0e49b4 to dd605cf Compare January 29, 2025 14:41
@sonarqubecloud
Copy link

@tonipel tonipel force-pushed the dont-disable-temporary-vehicle-immediatly branch from dd605cf to bbe654e Compare January 30, 2025 13:04
Copy link
Contributor

@mhieta mhieta left a comment

Choose a reason for hiding this comment

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

@tonipel Testsuite seems to fail still. Should be fixed before merge.

@tonipel tonipel force-pushed the dont-disable-temporary-vehicle-immediatly branch from e31376a to 7d3b87d Compare March 3, 2025 11:13
@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 3, 2025

@mhieta mhieta self-requested a review March 6, 2025 12:28
Copy link
Contributor

@mhieta mhieta left a comment

Choose a reason for hiding this comment

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

Looks good now

@mhieta mhieta merged commit 80e85c8 into develop Mar 6, 2025
3 checks passed
@mhieta mhieta deleted the dont-disable-temporary-vehicle-immediatly branch March 6, 2025 12:31
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.

2 participants