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.

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

Functions like createdOn.Date are ignored in GroupBy #226

Open
rivantsov opened this issue Jul 14, 2023 · 6 comments
Open

Functions like createdOn.Date are ignored in GroupBy #226

rivantsov opened this issue Jul 14, 2023 · 6 comments

Comments

@rivantsov
Copy link
Owner

See issue #225

@jasonlaw
Copy link

jasonlaw commented May 6, 2024

May I know will this be fixed as well? Thanks.

@rivantsov
Copy link
Owner Author

yes, it will be fixed as well, will try to focus on this and other general restructuring issues in the next few weeks. I will post more about the plans in a separate post.

@jasonlaw
Copy link

jasonlaw commented May 7, 2024

Thanks for the update, looking into the new release. :)

@jasonlaw
Copy link

jasonlaw commented May 7, 2024

I have one request that would like to get your attention.

Base on the discussion we have before at #150, may I know if the filtered relation supported in v1.9 before is part of the sql query or post filtering after retrieving from database?

If it is part of sql query, may I request the feature again? Here is my use case:

  1. A store have many members.
  2. The member can unregister themselves from the store, which will be marked as soft deleted.
  3. Those soft deleted members will be treated as "deleted record", and should not be retrieved in all the query.
  4. Now, for all the members relationship related query, we always need to add the IsDeleted checking explicitly. It would be nice if this can be handled implicitly by the filtered relation.

Thank you.

@rivantsov
Copy link
Owner Author

I will definitely look at that, no promises or dates yet, but I am looking at soft deletes

@rivantsov
Copy link
Owner Author

sorry for the delay but the problem #225 with expressions in GroupBy is fixed now

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

No branches or pull requests

2 participants