Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
This module will be available Drupal 8? I can help with is migration.
Comment | File | Size | Author |
---|---|---|---|
#11 | weather_module_port_to_d8-2467809-11.patch | 1.18 MB | tarasich |
Comments
Comment #1
toddy CreditAttribution: toddy commentedYes, I intend to start porting to D8 soon after the first release candidate of Drupal 8.
Comment #2
toddy CreditAttribution: toddy commentedComment #3
platinum1 CreditAttribution: platinum1 commentedAny thoughts on the D8 port, now that 8.0.0 is out?
Comment #4
teraflik CreditAttribution: teraflik commentedAny thoughts now? I am getting a hang of Drupal, and I think I might take this up.
Comment #5
toddy CreditAttribution: toddy commentedHi,
I currently lack the time to do a port of weather to Drupal 8, because so many things in Drupal have changed. I've started several times with a port and quickly discovered every time that I'm not able to simply port the module.
As far as I can see, the module would probably need a complete rewrite from scratch. And for this I simply do not have the time.
So, if you're willing to tackle this, please feel free to start a port.
Regards,
Tobias
Comment #6
joseph.olstadHi Toddy and everyone else, great module, for those interested,
there's a utility for helping with porting from previous versions of Drupal to D8 to make the job a lot easier .
The module is called drupalmoduleupgrader
it does quite a bit of the work for you, and provides code comments with api suggestions for stuff it can't do.
it's probably the best place to start at.
With that said, fantastic module, going to try it out now.
Comment #7
talenus CreditAttribution: talenus commentedHey all,
For a course I am taking I need to port a module from drupal 7 to drupal 8 as an assignment.
Looks like this weather module still needs to be ported? Has any progress been made? Has anyone tried using the "drupalmoduleupgrader"?
Comment #8
toddy CreditAttribution: toddy commentedHi,
as mentioned in #2467809-5: Porting module to Drupal 8 , this module is not easy to port to Drupal 8. So I would not advise to use this module as an assignment for a course. It's really difficult to port. Believe me.
The drupalmoduleupgrader also does not help much with porting this module, I've tried.
Anyway, please be my guest if you still want to port the module to D8 -- it would be much appreciated!
Regards,
Tobias
Comment #9
tarasichAttaching first version of D8 module port.
Features ported already:
Would be good to have followups for all other features not ported yet, including tests.
Comment #10
ysamoylenko CreditAttribution: ysamoylenko at EPAM Systems commentedHello @tarasich,
Thanks for your contribution.
I'm going to apply and test your patch, then I'll get back to you.
Comment #11
tarasichFixed bug with weather places not saved properly. Appeared after refactoring ;)
Attaching new patch, lets check this one.
Comment #12
ysamoylenko CreditAttribution: ysamoylenko at EPAM Systems commented@tarasich, thanks for your update.
I'll check the functionality again.
Comment #13
ysamoylenko CreditAttribution: ysamoylenko at EPAM Systems commented@tarasich, just re-tested the module functionality.
Everything works as expected.
I'm going to commit your changes and create the first module 'alpha' release for D8.
Great work!
Comment #15
ysamoylenko CreditAttribution: ysamoylenko at EPAM Systems commentedThe patch was committed.
Thanks, everyone for your participation in the module development.
Comment #16
joseph.olstadnicely done!