search

GreenMeteor / codebox

Allows you to add and use HTML snippets on your sidebar
3 stars 4 forks source link

Feature Request: Support adding multiple codeboxes #7

Closed dvn0 closed 4 months ago

dvn0 commented 4 months ago

Convenient module, but I want to be able to add multiple different codeboxes, so I can display multiple things.

ArchBlood commented 4 months ago

It may take some time before I can implement such a function, but I do like the idea.

dvn0 commented 4 months ago

If it wouldn't be too much overhead to provide a little guidance then I may be able to find time to work on this myself in a PR.

ArchBlood commented 4 months ago

If it wouldn't be too much overhead to provide a little guidance then I may be able to find time to work on this myself in a PR.

Thank you for the offer, currently to do something like this we'd need to update everything so that it's an ActiveRecord instead of a normal Model;

Adding to that we'd need the following;

ArchBlood commented 4 months ago

Okay, so here's the current state of this request, I've been trying to work on the implementation requested and here's what I've gotten done so far;

Screenshot_1 Screenshot_2

Note: Such changes requires much refactoring, and I'm not exactly sure if it's currently worth time implementing as a free service, I say this because this would also require more maintaining of code compared to the original intent of the module.

dvn0 commented 4 months ago

Looks great, that's pretty much exactly what I imagined. I like how you have the box titles.

I'm not exactly sure if it's currently worth time implementing as a free service, I say this because this would also require more maintaining of code compared to the original intent of the module.

You mean you should get paid for your work on this, or that the module shouldn't be free if it gets this feature?

ArchBlood commented 4 months ago

You mean you should get paid for your work on this, or that the module shouldn't be free if it gets this feature?

I try my best to keep as many modules free as possible, but maintaining the code is very time consuming, and I currently have little time at this moment due to other requests, I'll confirm that in v1.2.0 this will be implemented though.

dvn0 commented 4 months ago

Yea, understood. Sorry I couldn't be more helpful this time. Thanks much for your work.

ArchBlood commented 4 months ago

Yea, understood. Sorry I couldn't be more helpful this time. Thanks much for your work.

You can install the latest version from latest releases, or wait till HumHub v1.16.0 stable is released.

dvn0 commented 4 months ago

Thanks. I'll try it out on my test deployment now, and then wait for the HumHub release for production.