Copied to clipboard

Flag this post as spam?

This post will be reported to the moderators as potential spam to be looked at


  • Chris Kim 43 posts 212 karma points c-trib
    May 05, 2016 @ 04:29
    Chris Kim
    0

    Also import records

    I have extended the plugin to also import the form entries (records) and submitted a pull request: https://github.com/TimGeyssens/ContourToUmbracoFormsMigrator/pull/3

    The pull request also fixes an issue I had with the workflows - they were showing up and working, but could not be deleted due to the [workflowGUID].json file not having the correct form GUID set.

    Importing all the records makes a LOT of DB calls and is therefore rather slow. Scripting this is SQL would have probably been faster, but this way it's using the Umbraco.Forms.Core methods and someone else might be able to benefit from it too.

    Maybe Tim can improve on it, with access to the Umbraco.Forms source he'd have a better understanding of what goes on under the hood.

    Tim, regardless of whether or not you accept my pull request, it would be useful if you could update the plugin itself as that is still bundled with the original dll and does not include the fixes provided by jclementson

    Cheers

  • Chris Kim 43 posts 212 karma points c-trib
    Jun 15, 2016 @ 01:08
    Chris Kim
    0

    Have just added another fix that prevented the workflows to save correctly (now part of the same pull request)

  • Zac 238 posts 539 karma points
    Aug 10, 2016 @ 15:47
    Zac
    0

    Hi Chris,

    What Umbraco version are you upgrading to and what version of Umbraco.Forms.Core are you including in your project? We're trying to upgrade from v7.1.X with contour to v7.4.3 with Forms and we've tried using Contour 3.24 - 3.29 without any luck. We keep getting reference errors that

     'Umbraco.Forms.Core.FieldSet' does not contain a definition for 'Containers' and no extension method 'Containers' accepting a first argument of type 'Umbraco.Forms.Core.FieldSet' could be found (are you missing a using directive or an assembly reference?)    
    

    Thanks for any help

  • Zac 238 posts 539 karma points
    Aug 10, 2016 @ 16:33
    Zac
    0

    Silly mistake - we were trying to reference the old contour dll and need to reference the new Forms dll. Hooked this up with the Umbraco.Forms.Core dll from Forms 4.3.2 and it builds fine.

  • Chris Kim 43 posts 212 karma points c-trib
    Aug 11, 2016 @ 21:27
    Chris Kim
    0

    Good to hear you got it to work. I remember having an issue with the dll and I just added the Umbraco.Forms.Core NuGet package to the solution and changed the reference to that.

    We were upgrading Umbraco from 7.1.4 to 7.4.1 and then from the Contour 3.0.28 to Forms 4.2.1

    Let me know how you go.

  • Zac 238 posts 539 karma points
    Aug 15, 2016 @ 14:59
    Zac
    0

    Hey Chris,

    With the combination of your pull requests and a bunch of customizations that we made, we've been able to migrate very successfully so far. We have a ton of forms to migrate across several environments but things are looking great.
    I'm thinking about making a new package with these updates since it seems like this one isn't really supported anymore nor is it very functional in it's current state.

    Hope things went well for your migration.

  • Chris Kim 43 posts 212 karma points c-trib
    Aug 15, 2016 @ 23:45
    Chris Kim
    0

    Hi Zac,

    I am glad to hear that my pull requests were useful for you.

    Considering how long it's been since my pull request with no reaction from Tim, I agree that making a separate package probably makes sense.

    I really wish there was better official support for this migration since both Contour and Forms are official plugins but they just dropped one for the other. After all, these plugins are how Umbraco makes some of their money.

  • This forum is in read-only mode while we transition to the new forum.

    You can continue this topic on the new forum by tapping the "Continue discussion" link below.

Please Sign in or register to post replies