Skip to content

fix(backend): reset billed flag when report is moved to another project #692

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Apr 9, 2025
Merged

fix(backend): reset billed flag when report is moved to another project #692

merged 1 commit into from
Apr 9, 2025

Conversation

derrabauke
Copy link
Member

@derrabauke derrabauke self-assigned this Apr 9, 2025
@derrabauke derrabauke requested a review from a team as a code owner April 9, 2025 08:52
@derrabauke derrabauke requested a review from trowik April 9, 2025 08:54
@derrabauke derrabauke added the bug Something isn't working label Apr 9, 2025
MitanOmar
MitanOmar previously approved these changes Apr 9, 2025
View reviewed changes
trowik
trowik requested changes Apr 9, 2025
View reviewed changes
Copy link
Member

@trowik trowik left a comment •
edited
Loading

Choose a reason for hiding this comment

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

LGTM!

Could you update the snapshot?
docker compose exec backend pytest --snapshot-update

@derrabauke derrabauke merged commit 64b4725 into adfinis:main Apr 9, 2025
5 checks passed
@derrabauke derrabauke deleted the fix-billed-flag branch April 9, 2025 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@trowik trowik trowik approved these changes

@MitanOmar MitanOmar MitanOmar left review comments

@c0rydoras c0rydoras Awaiting requested review from c0rydoras

Assignees

@derrabauke derrabauke

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

bug: moved tasks keep their billed flag
3 participants
@derrabauke @MitanOmar @trowik