Date range table filter in Ant Design is broken

[StariMarco]

Date range table filter in Ant Design is broken

[alicanerdurmaz]

@StariMarco, thanks for the issue 👋 I will look into it

[frank-mendez]

I'm a newcomer to refine. I've managed to replicate the bug, and I'm now investigating it to create a solution as part of my exploration of the code. If that is okay? Thank you 😄

[alicanerdurmaz]

I'm a newcomer to refine. I've managed to replicate the bug, and I'm now investigating it to create a solution as part of my exploration of the code. If that is okay? Thank you 😄

Hello @frank-mendez, We are open to all contributions 🙌 Thanks for improving Refine. If you want I can assign this issue to you.

[rohitlohar45]

Hey @alicanerdurmaz I can also contribute to this issue, I have reproduced the issue and also looking for the fix. If you don't mind can you assign it to me?

[alicanerdurmaz]

Hey @alicanerdurmaz I can also contribute to this issue, I have reproduced the issue and also looking for the fix. If you don't mind can you assign it to me?

Hello @rohitlohar45, I am glad to hear that but since @frank-mendez wrote earlier, it would be better to assign it to him. Unfortunately, I replied to @frank-mendez too late, so he might have given up on doing the task. Let's wait until tomorrow and if we don't get an answer I'll be happy to assign it to you.

[rohitlohar45]

Hey @alicanerdurmaz I can also contribute to this issue, I have reproduced the issue and also looking for the fix. If you don't mind can you assign it to me?

Hello @rohitlohar45, I am glad to hear that but since @frank-mendez wrote earlier, it would be better to assign it to him. Unfortunately, I replied to @frank-mendez too late, so he might have given up on doing the task. Let's wait until tomorrow and if we don't get an answer I'll be happy to assign it to you.

No problem, actually have a workaround in my mind by back tracking the code. Would help even if I don't get assigned to this

[alicanerdurmaz]

@rohitlohar45 Thanks, I assigned it to you

[rohitlohar45]

I had a doubt that do refine handle the graphQl requests while filtering if so where it is handling it because the main issue is that in filters field the expected format is not sent and hence this error is received.

[frank-mendez]

Yes, I have also did some digging on the issue and indeed it was something to do with the format of the filters for graphql. This is what the filter looks like when we do onChange for <DatePicker.RangePicker />

{ "createdAt": { "between": { "lower": "2024-05-14T16:00:00.000Z", "upper": "2024-05-16T16:00:00.000Z" } } }

This is what happened to the filter for graphql when we refresh the page

{ "between": { "lower": { "$L": "en", "$d": "2024-05-14T16:00:00.000Z", "$y": "2024", "$M": "4", "$D": "15", "$W": "3", "$H": "0", "$m": "0", "$s": "0", "$ms": "0", "$isDayjsObject": "true" }, "upper": { "$L": "en", "$d": "2024-05-16T16:00:00.000Z", "$y": "2024", "$M": "4", "$D": "17", "$W": "5", "$H": "0", "$m": "0", "$s": "0", "$ms": "0", "$isDayjsObject": "true" } } }

My initial solution leads somewhere to format the filters in the defaultFilteredValue using the getDefaultFilter function. I'm not sure yet how to implement it since I'm also new graphql.

[alicanerdurmaz]

@frank-mendez, @rohitlohar45 Thanks for detailed explanation.

I think there is a problem in the process of encoding filters to string or parsing from string to filters. Code is here. I'll debug and let you know what I find.

[rohitlohar45]

@frank-mendez, @rohitlohar45 Thanks for detailed explanation.

I think there is a problem in the process of encoding filters to string or parsing from string to filters. Code is here. I'll debug and let you know what I find.

@alicanerdurmaz, Can you please check this PR: #5961 this solution is working fine, but not sure that have I maintained proper developer practices.