Skip to content

add index for classification #41111

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
Merged
nickvergessen merged 1 commit into from
Oct 31, 2023
Merged

add index for classification #41111

nickvergessen merged 1 commit into from
Oct 31, 2023

Conversation

@tcitworld
Copy link
Member

@tcitworld tcitworld commented Oct 25, 2023 •
edited
Loading

Summary

Replaces #35459
I also renamed the migration since it will go into 28, not 27, and bumped the app version.

Checklist

@tcitworld tcitworld added 3. to review Waiting for reviews feature: dav labels Oct 25, 2023
@tcitworld tcitworld added this to the Nextcloud 28 milestone Oct 25, 2023
@tcitworld tcitworld mentioned this pull request Oct 25, 2023
Closed
5 tasks
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Oct 27, 2023
@tcitworld tcitworld force-pushed the dav/classification-index branch from 2825a22 to 1663b9c Compare October 30, 2023 15:39
@tcitworld
Copy link
Member Author

tcitworld commented Oct 30, 2023

Rebased

@miaulalala
Copy link
Contributor

miaulalala commented Oct 31, 2023

Failing test is due to nextcloud/spreed#10814

Needs another rebase on master

@akhil1508 @tcitworld
add index for classification
439de11 
Signed-off-by: Akhil <[email protected]>
Signed-off-by: Thomas Citharel <[email protected]>
@tcitworld tcitworld force-pushed the dav/classification-index branch from 1663b9c to 439de11 Compare October 31, 2023 09:41
@nickvergessen nickvergessen merged commit 48bfbf8 into master Oct 31, 2023
@nickvergessen nickvergessen deleted the dav/classification-index branch October 31, 2023 12:56
@akhil1508 akhil1508 mentioned this pull request Mar 27, 2024
Closed
9 tasks
@joshtrichards joshtrichards mentioned this pull request Dec 22, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@st3iny st3iny st3iny approved these changes

@miaulalala miaulalala miaulalala approved these changes

@nickvergessen nickvergessen Awaiting requested review from nickvergessen

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

@kesselb kesselb Awaiting requested review from kesselb

Assignees

No one assigned

Labels

4. to release Ready to be released and/or waiting for tests to finish feature: dav

Projects

None yet

Milestone

Nextcloud 28

Development

Successfully merging this pull request may close these issues.

7 participants

@tcitworld @miaulalala @st3iny @nickvergessen @ChristophWurst @akhil1508