[BUG] Issue in Latest Post Section about wanderlust HOT 15 CLOSED

@krishnaacharyaa can I take this issue ?

from wanderlust.

@soheb-btw sure
Assigning it to you
Please let us know the time estimation to complete the task

from wanderlust.

@krishnaacharyaa I am unable to run recreate the bug on local, sorting seems to be fine on local.
while going through the code I have seen that all the posts that didn't had timeOfPost values in db would give you current time and date when api/posts is called.

from wanderlust.

Are there any post in db that don't have timeOfPost value ?

from wanderlust.

Good catch @soheb-btw , I suspect the problem is with the initial db values which is sample data.json from the readme file ....(ref)
Youu would may be able to reproduce there, if not .... Let me know I'll send the original db's json content....

I suspect that's the issue because I don't see timeofPost value in the sample data

from wanderlust.

I did import data from sample_posts.json while setting up the project and then removed all data for testing, both cases I was facing the date issue.

It will be great if you can send the original db's json and then I can debug further and we can know for sure.

from wanderlust.

Sure, tomorrow I'll send , please remind me if I forget, either here or in discord, tag master

from wanderlust.

Hi @soheb-btw, Please use this for reference, straight from the db, Let me know if you need more support, we can get on a call and resolve
wanderlust.posts.json

from wanderlust.

Hi @krishnaacharyaa missing timeOfPost values are creating the issue. It cannot be empty.

from wanderlust.

Sure @soheb-btw, thank you for the root cause. So were all do we have to change I guess the sample data and the db data right?

from wanderlust.

yes, we need add missing timeOfPost values in both files.

from wanderlust.

And one more change is required in /api/posts route, Assuming in All Posts section we are showing latest posts first, its sorting logic is not implemented for the route. I have made the code changes can I make a pull request for this ?

from wanderlust.

Sure, can you please follow the contributing guidelines and raise PR. @soheb-btw

from wanderlust.

@krishnaacharyaa I have created the pull request from a different account.

from wanderlust.

@soheb-btw I have addressed the PR, kindly make the changes and let me know

from wanderlust.