Direction Ascending not working

[ivanov]

Describe the bug

setting the ascending order direction only works on data loaded with descending order initially, and returns no results if it is the initial setting on page load.

Steps to reproduce

1. to go to https://demo.usememos.com/auth and login
2. click on the settings icon at the right edge of the "Search memos" input
3. change Direction to Ascending (the loaded memos are properly reordered.
4. observe that the URL is https://demo.usememos.com/?orderBy=asc and reload
5. no items show up, a spinner keeps spinning indefinitely.

The version of Memos you're using

demo

Screenshots or additional context

javascript console error:

app.Cqhkrlla.js:279 Uncaught (in promise) ClientError: /memos.api.v1.MemoService/ListMemos INVALID_ARGUMENT: failed to build find memos with filter: rpc error: code = InvalidArgument desc = invalid filter: found issue ERROR: <input>:1:1: undeclared reference to 'order_by_time_asc' (in container '')
 | order_by_time_asc == true
 | ^
    at s (app.Cqhkrlla.js:279:4865)
    at async f (app.Cqhkrlla.js:279:8975)
    at async Object.listMemos (app.Cqhkrlla.js:279:9383)
    at async Object.fetchMemos (app.Cqhkrlla.js:280:162390)
    at async c (app.Cqhkrlla.js:1157:4064)
    at async u (app.Cqhkrlla.js:1157:4376)


[johnnyjoygh]

Thanks for the information, I’ll handle this.

[johnnyjoygh]

Fixed with 849bf66, you can pull the stable image again to validate.