https://docs.all-hands.dev/ logo
Join Slack
Powered by
Pr to add inline LLM profile switch button <https...
# proj-agent
m
Pr to add inline LLM profile switch button https://github.com/OpenHands/OpenHands/pull/14301
🙌 1
c
actually i wonder if we can kill frontend/src/hooks/chat/use-model-interceptor.ts completely if we go with switch profile button 🤔
m
Do you mean: delete the command /model?
c
yep nod UI is more intuitive for doing this IMO
also meaning fewer code to maintain
m
I don't have strong opinion on that. I think I did the /model because was described in an issue in the sdk. I don't know if there are particular motivation for that... The only one I have in mind is programmatically changing the model in a convo
c
sg... in that case maybe we will keep it for now (until it is breaking 😅 )
can you also get this PR to go past all-hands-bot though?
ie have bot approves first, since i'm not 100% confident in my sensitivety to FE code changes lolsob
m
It is already 🙂
👍 1
c
sg! LGTM
m
I ask for review only after all-hands-bot is happy 😉
c
looking pretty straghtfoward for me
do you want me to merge it?
m
yes please
c
done!
ty 1
2 Views
Open in Slack
PreviousNext
Powered by