What are the Slack Archives?

It’s a history of our time together in the Slack Community! There’s a ton of knowledge in here, so feel free to search through the archives for a possible answer to your question.

Because this space is not active, you won’t be able to create a new post or comment here. If you have a question or want to start a discussion about something, head over to our categories and pick one to post in! You can always refer back to a post from Slack Archives if needed; just copy the link to use it as a reference..

Is it correct that you can only unpublish but not delete CMS pages and blocks in the Zed UI?

UM4BZSK7T
UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet

Is it correct that you can only unpublish but not delete CMS pages and blocks in the Zed UI?

Comments

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    True.

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    There is a facade method for deletion though in the code.

  • UM4BZSK7T
    UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet

    Thanks for the answer. πŸ™‚

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    But the method is broken a bit in the latest versions. It's known already and will be fixed. :)

  • UM4BZSK7T
    UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet

    The fix is just to add the cascade on delete. <foreign-key name="spy_cms_page_store-fk_cms_page" foreignTable="spy_cms_page" onDelete="CASCADE"/> I would make a pull request easily but as far as I know you won't accept any from contrib...

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Not that we won't but we can't just merge a PR directly into the module as modules are being released by our release bot. But you can create a PR of course. This only means that I'll need to create an internal ticket out of it to follow our development\release process.

  • UM4BZSK7T
    UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet

    Oh okay cool. So against those single repos in this case spryker/cms?

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Seems so.

  • UM4BZSK7T
    UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet
  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Thanks, I will process it. πŸ‘

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    Ok, our CMS techlead says that this isn't enough and this would be major in this case.

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice

    So we will fix it in other way and I will close this PR...

  • UM4BZSK7T
    UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet

    That is sad to hear. I really need to delete CMS Pages and it currently works with that added. If there is another problem inside of the API better said the PageRemover() class it is currently not affecting me.

  • UM4BZSK7T
    UM4BZSK7T Posts: 174 πŸ§‘πŸ»β€πŸš€ - Cadet

    And about the pull request. You will have to add this anyways or the deletion cascade will not work...

  • UKTSRTD5M
    UKTSRTD5M Posts: 77 πŸ§‘πŸ»β€πŸš€ - Cadet

    Hi @UK9N7MP96, can you pls connect us with the techlead mentioned so that we can see what the major problem is and if/how can contribute . We need this to get our CMS integration to work properly. Thanks in advance.

  • Valerii Trots
    Valerii Trots SRE @ Spryker Sprykee Posts: 1,654 ✨ - Novice
  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 πŸ§‘πŸ»β€πŸš€ - Cadet

    Hi guys, this bugfix is currently in progress. Do you want to know the direction we gonna use earlier?

  • UKTSRTD5M
    UKTSRTD5M Posts: 77 πŸ§‘πŸ»β€πŸš€ - Cadet

    Hi Stanislav, thanks for you quick answer. Do you have a rought estimation when you expect the fix to be available?

  • Stanislav Matveyev
    Stanislav Matveyev Tech Lead @ Spryker Sprykee Posts: 211 πŸ§‘πŸ»β€πŸš€ - Cadet

    most possible next week

  • UKTSRTD5M
    UKTSRTD5M Posts: 77 πŸ§‘πŸ»β€πŸš€ - Cadet

    very nice πŸ™‚

  • UKTSRTD5M
    UKTSRTD5M Posts: 77 πŸ§‘πŸ»β€πŸš€ - Cadet

    Thanks for keeping us up to date