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

Feature/content to embedding model #18

Merged
merged 19 commits into from
Oct 25, 2024
Merged

Feature/content to embedding model #18

merged 19 commits into from
Oct 25, 2024

Conversation

sudan45
Copy link
Collaborator

@sudan45 sudan45 commented Sep 17, 2024 •
edited
Loading

Addresses

Changes

  • Detailed list or prose of changes
  • Breaking changes
  • Changes to configurations

This PR doesn't introduce any:

  • temporary files, auto-generated files or secret keys
  • n+1 queries
  • flake8 issues
  • print
  • typos
  • unwanted comments

This PR contains valid:

  • tests
  • permission checks (tests here too)

@sudan45 sudan45 marked this pull request as draft September 17, 2024 09:02
@sudan45 sudan45 force-pushed the feature/content-to-embedding-model branch from 70ade3e to 65c2884 Compare September 18, 2024 08:40
@sudan45 sudan45 marked this pull request as ready for review September 18, 2024 08:50
thenav56
thenav56 approved these changes Sep 18, 2024
View reviewed changes
.flake8 Outdated Show resolved Hide resolved
main/settings.py Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
docker-compose.yml Outdated Show resolved Hide resolved
content/tasks.py Outdated Show resolved Hide resolved
content/signals.py Outdated Show resolved Hide resolved
content/tasks.py Outdated Show resolved Hide resolved
content/tasks.py Show resolved Hide resolved
content/views.py Outdated Show resolved Hide resolved
@thenav56 thenav56 self-requested a review September 18, 2024 09:12
@sudan45 sudan45 force-pushed the feature/content-to-embedding-model branch from 89759ba to dcec82c Compare September 27, 2024 04:58
@sudan45 sudan45 force-pushed the feature/content-to-embedding-model branch from dcec82c to dc621cf Compare September 30, 2024 08:56
@sudan45 sudan45 force-pushed the feature/content-to-embedding-model branch from 1912d74 to fc9cc51 Compare October 15, 2024 06:17
@thenav56 thenav56 merged commit 174c232 into develop Oct 25, 2024
5 checks passed
@thenav56 thenav56 deleted the feature/content-to-embedding-model branch October 25, 2024 04:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thenav56 thenav56 Awaiting requested review from thenav56

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
@sudan45 @thenav56