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.

Sign up for GitHub

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

Jump to bottom

Search combined #5271

Draft
wants to merge 82 commits into
base: main
Choose a base branch
from
Draft

Search combined #5271

wants to merge 82 commits into from

Conversation

dessalines
Copy link
Member

WIP

dessalines and others added 30 commits November 2, 2024 13:04
@dessalines
Renaming person_mention to person_comment_mention.
5b87cd8
@dessalines
Finishing up post body mentions.
6a7b1d4
@dessalines
Combined tables try 2
39b3ec2
@dessalines
Finishing up combined report table.
1ff8ae0
@dessalines
Fix ts optionals.
2351c7a
@dessalines
Adding tests, triggers, and history updates for report_combined.
0ba961f
@dessalines
Adding profile.
d8dda44
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
d68c0a6
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
a9f28af
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
3adadf9
@dullbananas
Add cursor pagination to report_combined view (#5244)
588e1f6 
* add pagination cursor

* store timestamp instead of id in cursor (partial)

* Revert "store timestamp instead of id in cursor (partial)"

This reverts commit 89359dd.

* use paginated query builder
@dessalines
Merge branch 'combined_tables_2' of https://github.com/LemmyNet/lemmy …
f25d346 
…into combined_tables_2
@dessalines
Fixing migration and paged API.
7fdbb58
@dessalines
Using dullbananas trigger procedure
00beccd
@dessalines
Removing pointless list routes, reorganizing tests.
1776de2
@dessalines
Fixing column XOR check.
612c2e9
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
5a8b9de
@dessalines
Forgot to remove list report actions.
0c854d2
@dessalines
Cleanup.
802a8a6
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
4674821
@dessalines
Merge branch 'main' into combined_tables_2
91ca6d3
@dessalines
Use internal tagging.
4cac67c
@dessalines
Merge branch 'combined_tables_2' of https://github.com/LemmyNet/lemmy …
83988cd 
…into combined_tables_2
@dessalines
Fixing api tests.
82a14fc
@dessalines
Adding a few indexes.
1f28407
@dessalines
Fixing migration name.
34a440a
@dessalines
Fixing unique constraints.
1a739a1
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
e1affa8
@dessalines
Addressing PR comments.
921d532
@dessalines
Start working on profile combined
724856d
@dessalines
Using dullbananas trigger procedure
63b7f72
@dessalines
Merge remote-tracking branch 'origin/main' into combined_modlog
5c2205a
@dessalines
Merge branch 'combined_profile' into combined_modlog
6363e7d
@dessalines
Adding the full combined view queries.
9d36f1f
@dessalines
Adding tests.
c157b8f
@dessalines
taplo fmt.
cdd5f65
@dessalines
Upgrading package.json deps.
6318cac
@dessalines
Updating pnpm
b3b8efd
@dessalines
Merge branch 'post_body_mentions' into combined_inbox
f133079
@dessalines
Most of the bulk work done, need to add tests yet.
05f218d
@dessalines
Finishing up inbox.
41cfdca
@dessalines
Using assert_length
c8a2652
@dessalines
Fixing sql_format.
37aa200
@dessalines
Running fmt.
ec3dcb7
@dessalines
Fixing cargo shear.
216d089
@dessalines
Fixing clippy.
aa9c140
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
1e6b3fb
@dessalines
Addressing PR comments.
c72d5e8
@dessalines
Merge remote-tracking branch 'origin/main' into combined_modlog
942da64
@dessalines
Merge branch 'combined_modlog' into combined_inbox
17d2de8
@dessalines
Starting to work on search combined.
dada964
@dessalines
Fix
031dc26
@dessalines
Removing serialization
7c962d6
@dessalines
Merge remote-tracking branch 'origin/main' into combined_tables_2
ae9da4d
@dessalines
Moving db_views_actor and _moderator into db_views.
3721067 
- This is necessary because the combined views use both, and that
  separation was arbitrary to begin with. db_schema has no such crate
  separation.
@dessalines
Merge branch 'combined_tables_2' into search_combined
fb1f2ff
@dessalines
Adding search combined view, need to write tests yet.
4586947
@dessalines
Filters done, working on tests.
3890834
@dessalines
Adding tests for person, post, and community.
48283a1
@dessalines
Finishing up tests.
9cb28e7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@phiresky phiresky Awaiting requested review from phiresky phiresky will be requested when the pull request is marked ready for review phiresky is a code owner

@dullbananas dullbananas Awaiting requested review from dullbananas dullbananas will be requested when the pull request is marked ready for review dullbananas is a code owner

@Nutomic Nutomic Awaiting requested review from Nutomic Nutomic will be requested when the pull request is marked ready for review Nutomic is a code owner

@SleeplessOne1917 SleeplessOne1917 Awaiting requested review from SleeplessOne1917 SleeplessOne1917 will be requested when the pull request is marked ready for review SleeplessOne1917 is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dessalines @dullbananas