Add group filter to stories

[YaleChen299]

Missing Permission check.
Current behaviour of the model function is:

• if groupID is given, filter by groupID,
• if the groupID is nil, gets all stories.

In the controller, the current routing and middle ware guarantees that the list all route will have a valid groupID.

Part of #15.

[github-actions]

coverage: 49.36% (-1.3%) from 50.642% when pulling 2741bea on add_group_filter_to_stories into ba486fb on main.

[RichDom2185]

Blocked by #97 (depends on the same migration file).

[RichDom2185]

Direction looks good. Just one question: does setting to nil actually filter WHERE group_id IS NULL? Or does it simply ignore the field, can we test this behaviour?

[RichDom2185]

Direction looks good. Just one question: does setting to nil actually filter WHERE group_id IS NULL? Or does it simply ignore the field, can we test this behaviour?

I'm asking because when setting values, we need to explicitly use an NULL expression in gorm (nil fields are simply ignored, e.g. see #85).

[YaleChen299]

Direction looks good. Just one question: does setting to nil actually filter WHERE group_id IS NULL? Or does it simply ignore the field, can we test this behaviour?

I'm asking because when setting values, we need to explicitly use an NULL expression in gorm (nil fields are simply ignored, e.g. see #85).

I think i have tests for that, when i use the struct where, with group id as null, it will explicit find those stories without groups.

Actually, let me refine the tests. I might not cover the case well.

Edit: passing nil will get all the stories.

[RichDom2185]

Edit: passing nil will get all the stories.

Ok, I will update the PR description.

[RichDom2185]

LGTM for now, thanks!