Hylozoic / hylo-evo

Hylo UI
Apache License 2.0
36 stars 10 forks source link

Reload current user whenever looking at the group detail page, in cas… #1666

Closed thomasgwatson closed 3 months ago

thomasgwatson commented 3 months ago

…e membership has changed

closes #1663

thomasgwatson commented 3 months ago

We could add a similar refetch in the group welcome modal as well, as a

I tried accessing the group in a range of ways (clicking on the notif, group explorer) and those didn't get to me to place where 1) I had a membership 2) but the membership didn't have a joinQuestionsAnsweredAt value

On Tue, 30 Jul 2024 at 18:04, Tibet Sprague @.***> wrote:

@.**** approved this pull request.

seems fine, and this fixes the double answering of questions? seems like that happens when going to the group home/stream page, not group details. but this happens first?

— Reply to this email directly, view it on GitHub https://github.com/Hylozoic/hylo-evo/pull/1666#pullrequestreview-2208955511, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABHT4AUMHN3POQS6K3R6MS3ZPAZYXAVCNFSM6AAAAABLXNVBJCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEMBYHE2TKNJRGE . You are receiving this because you authored the thread.Message ID: @.***>

thomasgwatson commented 3 months ago

(And when you don;t have a group membership, it routes you to GroupDetail

On Tue, 30 Jul 2024 at 19:21, Thomas Gabriel Watson < @.***> wrote:

We could add a similar refetch in the group welcome modal as well, as a

I tried accessing the group in a range of ways (clicking on the notif, group explorer) and those didn't get to me to place where 1) I had a membership 2) but the membership didn't have a joinQuestionsAnsweredAt value

On Tue, 30 Jul 2024 at 18:04, Tibet Sprague @.***> wrote:

@.**** approved this pull request.

seems fine, and this fixes the double answering of questions? seems like that happens when going to the group home/stream page, not group details. but this happens first?

— Reply to this email directly, view it on GitHub https://github.com/Hylozoic/hylo-evo/pull/1666#pullrequestreview-2208955511, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABHT4AUMHN3POQS6K3R6MS3ZPAZYXAVCNFSM6AAAAABLXNVBJCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDEMBYHE2TKNJRGE . You are receiving this because you authored the thread.Message ID: @.***>

tibetsprague commented 3 months ago

as long as it fixes the specific case Krisha ran into, which I have also seen then this is good enough for now