Hey :wave:
Could someone review this PR which ena...
# proj-agent-canvasm
modern-fireman-99755
04/27/2026, 5:24 PMHey 👋
Could someone review this PR which enables LLM profiles?
https://github.com/OpenHands/OpenHands/pull/14146
It is just the BE part, the FE will come in a next one.
(cc. @quick-window-91705)
👀 1
ty 1
q
quick-window-91705
04/27/2026, 5:57 PMLGTM overall
Left a comment using OH regarding persistence in saas
👍 1
quick-window-91705
04/27/2026, 6:16 PMAlso lmk if you'd like an enterprise deployment to test your changes (I think we don't allow it from forks but I can find a temp workaround)
e
elegant-branch-54423
04/27/2026, 6:47 PMThanks Vasco! FYI, I know FE is a separate ticket, but @breezy-kilobyte-39430 and @hundreds-twilight-32833 have collaborated on a FE design for this. Just wanted to make sure it was on your radar 🙏
m
modern-fireman-99755
04/27/2026, 6:48 PM@elegant-branch-54423 yes i have already some feedbacks from them. And i will tag both once i open the FE Pr ;)
🙌 1
thanks on a stick 1
modern-fireman-99755
04/28/2026, 9:08 AM@quick-window-91705 I worked on the PR. Can you give another look?
And thanks for your comment 🙂 I totally forgot about saas and enterprise.
About trying it in staging. I think once the FE is also merged it will be much easier to test it. I don't know if it is worth to do that now as it is just BE. But as you want.
FYI: I have the FE PR already in draft. I will open it once the BE one is merged 😉
🙌 1
q
quick-window-91705
04/28/2026, 2:34 PMLGTM
Locking in saas seems like a problem because of multiple workers; I've approved the PR though as we can figure it out in a future PR
For FE it may be worth spinning up a feature env, will decide based on the changes in the PR!
ty 1
m
modern-fireman-99755
04/28/2026, 3:01 PMI answered your comments and made the relative changes.
I don't have write permissions, i.e., if you are happy please merge...
✅ 1
2 Views