Skip to content

[-] Ignore some code to working if there is order_with_respect_to#4

Open
HuyNguyenDinh wants to merge 2 commits intorozumdev:masterfrom
HuyNguyenDinh:master
Open

[-] Ignore some code to working if there is order_with_respect_to#4
HuyNguyenDinh wants to merge 2 commits intorozumdev:masterfrom
HuyNguyenDinh:master

Conversation

@HuyNguyenDinh
Copy link

Hi, looks like the code in model.save method not working well if using with "order_with_respect_to".
I think that we need to temporary disable it if there no plan to implement the ScheduleBalancing. We can handle it in the Django pre_save signal

@HuyNguyenDinh HuyNguyenDinh changed the title [-] Ignore some code to lib work if there is order_with_respect_to [-] Ignore some code to working if there is order_with_respect_to Dec 16, 2024
@rozumalex
Copy link
Contributor

Hi @HuyNguyenDinh
Could you please add a detailed description of the problem so we can address it properly?

@HuyNguyenDinh
Copy link
Author

Hi @rozumalex ,
I found out that using order_with_respect_to it set the rank to None. So the pre_save of RankField was change it to inital behavior, this behavior make it always change to the top (or bottom base on the insert_to_bottom) instead of the correct position.

@rozumalex
Copy link
Contributor

Hey! @HuyNguyenDinh I still remember about your problem, just a bit busy at my work right now. I will try to give it a look soon, as I feel more changes are required here.

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