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

[GAL-4030] Exclude events for curated feed #1157

Merged
merged 1 commit into from
Aug 17, 2023
Merged

Conversation

jarrel-b
Copy link
Collaborator

@jarrel-b jarrel-b commented Aug 17, 2023

Excludes events from the curated feed if posts are enabled for the user, but will still display events if posts are not enabled. The trending feed will continue to show events, @kaitoo1 should we start excluding events from the trending feed too? We revert to the trending feed if the user isn't logged in.

@jarrel-b jarrel-b requested a review from kaitoo1 August 17, 2023 18:51
@github-actions github-actions bot added the query label Aug 17, 2023
@jarrel-b jarrel-b changed the title Exclude events for curated feed [GAL-4030] Exclude events for curated feed Aug 17, 2023
@jarrel-b jarrel-b merged commit d61f60b into main Aug 17, 2023
7 checks passed
@jarrel-b jarrel-b deleted the jarrel/gal-4030-posts-only branch August 17, 2023 22:02
benny-conn added a commit that referenced this pull request Aug 21, 2023
* deleted comments and reply pagination

* Include only posts (#1157)

* typo

* remove unused comment queries

* fix reply to and count replies

* migration for updating comment indicies

* removed comments

---------

Co-authored-by: jarrel <[email protected]>
benny-conn added a commit that referenced this pull request Oct 3, 2023
* initial implementation

* correct subject, post mentions

* resolve community mention correctly

* initial comment response

* deleted comments and paginated replies (#1161)

* deleted comments and reply pagination

* Include only posts (#1157)

* typo

* remove unused comment queries

* fix reply to and count replies

* migration for updating comment indicies

* removed comments

---------

Co-authored-by: jarrel <[email protected]>

* Benny/mention data new (#1170)

* mention data

* mention entity

* dummy commit

* mentions table

* notification

* rename migration, fix queries, rename index

* fix conflicts

* dummy commit

* migration numbers

* push notifications for new events

* skip replyto

---------

Co-authored-by: jarrel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant