Skip to content

Conversation

@marcelsavegnago
Copy link
Member

No description provided.

@marcelsavegnago marcelsavegnago changed the title [16.0][FIX] ai_oca_bridge_mrp: change icon module (get icon from ai_oca_bridge) and fix module name on manifest [16.0][FIX] ai_oca_bridge_mrp: change module icon (get icon from ai_oca_bridge) and fix module name on manifest Sep 20, 2025
@arielbarreiros96
Copy link
Contributor

arielbarreiros96 commented Sep 24, 2025

Hello @marcelsavegnago I'm not sure if it really makes sense to change the module icon to the same one from the bridge itself.
In any case please check on #41 , it is probably better to handle models trough config instead of keep creating glue models. If that PR gets merged will effectively render obsolete all these glue modules

@marcelsavegnago
Copy link
Member Author

Hello @marcelsavegnago I'm not sure if it really makes sense to change the module icon to the same one from the bridge itself. In any case please check on #41 , it is probably better to handle models trough config instead of keep creating glue models. If that PR gets merged will effectively render obsolete all these glue modules

Hello @arielbarreiros96

I totally agree with you about the icon — no need to duplicate the bridge one.

Regarding PR #41, I’ve been following it too and had the same thoughts about the current architecture and how to avoid building glue modules. Big congrats on the initiative, I’m really rooting for it here! :D

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.

3 participants