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

Decouple CdGpt getAnalysisContent method from entryAnalysisSchema model methods. #173

Open

hiyaryan opened this issue Oct 4, 2024 · 1 comment

Assignees

Labels

dev refactor todo

Member

hiyaryan commented Oct 4, 2024 •

edited

Loading

Accomplish TODO below from commit in backend/src/models/entry/entryAnalysis.ts of #169 and remove ESLint disable comments.

Line: 58
Message: // TODO: pull out this method to somewhere else. dependency on CdGpt not great
Commit: 6007601

hiyaryan assigned davidzlu

hiyaryan added dev todo refactor labels

hiyaryan mentioned this issue

refactor!: migrate models to TS with unit tests #169

Merged

davidzlu mentioned this issue

refactor: move calls to LLM into CdGpt services #196

Draft

Collaborator

davidzlu commented Nov 23, 2024

EntryServcies._updateEntry would need to be updated as well

davidzlu mentioned this issue

refactor!: create service layers for create, update, and delete operations #195

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment