Skip to content

edit tiny bug, finished replacing completions_v2 to completions#164

Merged
FranardoHuang merged 5 commits intomainfrom
merge-comp
Oct 15, 2025
Merged

edit tiny bug, finished replacing completions_v2 to completions#164
FranardoHuang merged 5 commits intomainfrom
merge-comp

Conversation

@benjaminxu117
Copy link
Collaborator

@benjaminxu117 benjaminxu117 commented Oct 13, 2025

after testing completions_v2, we are good to replace v2 to original completions.
also fixed a tiny bug in UserFocus. Bug happens if params doesn't contain user_focus attribute, original v2 logic will pass in an empty UserFocus() object into function, but UserFocus model doesn't have a default value for file_uuid.

@FranardoHuang FranardoHuang merged commit e0f5c16 into main Oct 15, 2025
1 check failed
@FranardoHuang FranardoHuang deleted the merge-comp branch October 15, 2025 04:57
FranardoHuang pushed a commit that referenced this pull request Mar 12, 2026
* edit tiny bug, finished replacing completions_v2 to completions

* remove unused model

* add default param to answer and problem content

* remove depracated code, refractor user focus

* pack get problem content

---------

Co-authored-by: = <=>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants