Skip to content
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

[Fix] delivery_auto_refresh manually delete delivery line #780

Closed
wants to merge 1 commit into from
Closed

[Fix] delivery_auto_refresh manually delete delivery line #780

wants to merge 1 commit into from

Conversation

santostelmo
Copy link
Contributor

Context

When deleting manually delivery line we get the following:
delivery_line_delete_error

@santostelmo santostelmo force-pushed the 16.0-fix-delivery_auto_refresh-unlink-delivery-line branch from dde0c86 to 576ae8b Compare March 12, 2024 09:50
@santostelmo
Copy link
Contributor Author

@rousseldenis the unit test self describes the issue that we are facing

@renda-dev renda-dev mentioned this pull request Mar 27, 2024
Merged
@santostelmo
Copy link
Contributor Author

Closed this PR. After a second thought I think we are not suppose to delete delivery line.
In the other hand we should not face this error if we try todo it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rousseldenis rousseldenis rousseldenis left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@santostelmo @rousseldenis