PerplexDigital / Perplex.ContentBlocks

Block based content editor for Umbraco
MIT License
31 stars 15 forks source link

Is the latest version compatible with v13? #84

Closed PanktiBodiwala closed 8 months ago

PanktiBodiwala commented 8 months ago

Hi Team,

I'm upgrading my project from v11.4.2 to v13. I upgraded perplex to the latest version (3.0) however after an upgrade, every block is showing an error 'Block definition not found'. Neither does it allow me to add a new content item, instead showing an error 'No Content Block definition found'.

I wonder if it is because of the Nested Content being deprecated in v13?

Please let me know if you need any further info.

Thanks

PerplexDaniel commented 8 months ago

We do not see these issues in v13, but we do set ShowDeprecatedPropertyEditors to true in appsettings. You can try to see if this solves the issue.

"Umbraco": {
  "CMS": {
    "Content": {      
      "ShowDeprecatedPropertyEditors": true
    },
  }
}

Can you also check the response to /umbraco/backoffice/api/contentblocksdefinitionapi/GetDefinitionsForPage in the backoffice when you are on a page that contains ContentBlocks? It should return an array of definitions. If not, please check your code if you are actually still adding definitions to the repository using IContentBlockDefinitionRepository.Add(...) somewhere in your code.

Nothing fundamental has been changed since v2 around any of this so I wonder what's going wrong for you.

PanktiBodiwala commented 8 months ago

Hi Daniel,

Thank you for the guidance. It is working now.

Out of curiosity, are you planning to introduce a migrator that can convert the NC blocks to the BLE? Would be extremely helpful in the future when deprecated property editors would not be available at all.

Kind Regards

PerplexDaniel commented 8 months ago

Good to hear it works.

are you planning to introduce a migrator that can convert the NC blocks to the BLE?

While I cannot make any promises it is likely that for the next Umbraco LTS we at Perplex also need to migrate to either the native Block List or migratie ContentBlocks itself to use BlockList as base rather than NestedContent. Either way we probably need some sort of data migration from NC -> BL, so I do expect some sort of Perplex.ContentBlocks.DataMigration package that will migrate data in some way (either to standalone BlockList types or to ContentBlocks with BlockList as underlying data model).

Not sure though, and regardless this will probably only be released around the time v17 (next LTS) is out and not when v14 (first major without NestedContent) is released since it's not LTS. v13 will be supported longer than all of v14 - v16.

Hope this gives you enough info, closing this now since your issue has been resolved!

PanktiBodiwala commented 8 months ago

Thanks Daniel, this makes sense.