backdrop-ops / contrib

Apply to join the contributed code developer team.
23 stars 16 forks source link

Port request for OwlCarousel #450

Open djzwerg opened 4 years ago

djzwerg commented 4 years ago

Name of the module, theme, or layout OwlCarousel

Link to the drupal.org module, theme, or layout https://www.drupal.org/project/owlcarousel

rudy880719 commented 3 months ago

Hello, I want to migrate this module to backdrop, but I see that there are dependencies with the jquery_update and variable modules, these do not exist for backdrop, could you give me some guidance on this. It is also necessary to migrate these modules as well.

djzwerg commented 3 months ago

I think these modules won't bei needed in Backdrop CMS.

djzwerg commented 3 months ago

It looks like that OWL Carousel is dead. On their page there is noted that Tiny Slider should be used (https://github.com/ganlanyuan/tiny-slider). But there is no D7 module for this.

rudy880719 commented 3 months ago

@djzwerg I don't think so, Did you see the module page. https://www.drupal.org/project/owlcarousel, Why is dead?. I used it frequently

argiepiano commented 3 months ago

jquery_update is not used in BAckdrop CMS. It was used in D7 to provide updates to jquery included in core. In Backdrop, we are currently using jquery 3, which is the latest major version.

As for the variable module - this would depend on how it's used in the D7 module. I see that variable provides metadata for D7 variables, allowing things such as access callbacks and tokens. Since Backdrop deprecated D7-like variables, then you would probably need to find a way to replicate those additional functionalities for json configurations - but it may not be necessary...

djzwerg commented 3 months ago

@djzwerg I don't think so, Did you see the module page. https://www.drupal.org/project/owlcarousel, Why is dead?. I used it frequently

The GitHub project is dead since six years (pls see https://github.com/OwlCarousel2/OwlCarousel2?tab=readme-ov-file). So there are no security patches or issue fixes anymore.

rudy880719 commented 3 months ago

@djzwerg @argiepiano I came to this community to help. What other module do you think is a high priority to migrate?

djzwerg commented 3 months ago

I really love this module but I don't know how to maintain issues e.g. within the library. Maybe there is a chance to build a Module for Tiny Slider?

rudy880719 commented 3 months ago

My apologies @djzwerg , You are right. I did't see the github page.

rudy880719 commented 3 months ago

I really love this module but I don't know how to maintain issues e.g. within the library. Maybe there is a chance to build a Module for Tiny Slider?

I will check it.. thanks

rudy880719 commented 3 months ago

@djzwerg @argiepiano @laryn I have good news, I have built a version of Tiny Slider for backdrop. As we know OwlCarousel is dead, and this is an alternative, I will appreciate if you review it and send me your comments and suggestions.

rudy880719 commented 3 months ago

@djzwerg Hello, what is the step to publish this module in the community?

argiepiano commented 3 months ago

@rudy880719 are you a member of the contrib team?

https://docs.backdropcms.org/documentation/contribute-to-a-module-layout-or-theme https://docs.backdropcms.org/documentation/contrib-application-process

argiepiano commented 3 months ago

I've taken a quick look at your repo of Tiny Slider.

stpaultim commented 3 months ago

@rudy880719 If we have not met before, welcome to the community and thanks for working on this module.

I just tested it out and I love it. Very simple with basic functionality and it looks good. I have a non-profit site that needs a module like this one right now. We'll test it there as well.

https://github.com/rudy880719/tiny_slider

rudy880719 commented 3 months ago

@argiepiano Thank you for the code suggestions, and for the link to be a member of the community, in fact I'm not one yet, I appreciate your effort to review and provide corrections f or the code, it's very important to me those suggestions. (Issues were fixed)

rudy880719 commented 3 months ago

@rudy880719 If we have not met before, welcome to the community and thanks for working on this module.

I just tested it out and I love it. Very simple with basic functionality and it looks good. I have a non-profit site that needs a module like this one right now. We'll test it there as well.

https://github.com/rudy880719/tiny_slider

@stpaultim Any time, thank you for the words, I appreciate it. I will continue working in this module, I want to add a formatter field to Entity Reference and Terms.

stpaultim commented 3 months ago

@rudy880719

My colleagues looked at your module and like it better than the one we are using for a new site we are building right now. We'll likely use it on that project. Looking forward to an official release.

stpaultim commented 2 months ago

@rudy880719 We're testing the Tiny Slider on a dev site in 2 places:

We won't be able to use it when we launch unless it's on official contrib project. If you need any help moving it into contrib, please ask. I'm happy to help out in any way possible or to answer any questions you have.

rudy880719 commented 2 months ago

Hello @stpaultim , I will need your help to move tiny slider and view timeline into contrib. Would you like to explain me how is the process. I have been busy, and I forgot it.

argiepiano commented 2 months ago

@rudy880719 see if this post helps you transfer the project to backdrop-contrib.

rudy880719 commented 2 months ago

@argiepiano @stpaultim project transfered to backdrop-contrib and created new release https://backdropcms.org/project/tiny_slider Thank you very much