umbraco / Umbraco.Deploy.Issues

1 stars 0 forks source link

Image alt text not transferred. Deploy bug issue! #174

Closed Jay-1975 closed 1 year ago

Jay-1975 commented 1 year ago

A brief description of the issue goes here.

Issue with alt text data not deploying across dev->stage->Live environments. Anders from Umbraco support has tested our issue with the same result as we have.

Reproduction

If you're filing a bug, please describe how to reproduce it. Include as much relevant information as possible, such as:

Bug summary

Open up image in content pic01 , Edit image within content page to include alt text. Save and Publish and deploy changes to Staging then Live from Dev. Text in the alt text field is not transferred between environments.

Specifics

Occurs in Dev and Staging Environments. There is no error, the text simply doesn't transfer over to the next environment.

Steps to reproduce

Expected result

Expect to see text from image in content page transferred from dev to stage .

Actual result

Alt text data not showing in the next environment (in this case staging). Also, doing the same from staging to Live produces the same result (no text in alt text).


_This item has been added to our backlog AB#31871_

AndyButland commented 1 year ago

I think this may be expected... as you've edited the media item here, and only chosen to transfer the content. When content is transferred it will transfer over any related media that doesn't exist in the target environment, but not any that has changed.

Could you try also selecting the media and transferring that please, and confirm that you see the updated alt text in the target environment?

If not, then there is an issue. If so, then I think that's expected... will then have a think about whether this behaviour should change, but I feel it may be necessary for performance reasons (in that content deployments would take longer if they also had to ensure all related media matches).

Jay-1975 commented 1 year ago

Hi Andy,

Thanks for your reply. We will give your suggestion a try and let you know ASAP.

Jay-1975 commented 1 year ago

Just heard back from Montreal Team:

OK. I synced the media first.

Checked. alt text not transferred.

I synced the content block afterwards.

Checked. Same results.

as expected, alt text not transferred between servers

Jay-1975 commented 1 year ago

the JSON data transferred/saved does not appear to mirror the source

AndyButland commented 1 year ago

OK, will see if I can replicate. What data type/property editor is used for that image field I can see in the screenshot please? And what version of the CMS and Deploy is being used?

Jay-1975 commented 1 year ago

It is utilizing the image cropper and the Image media type.

We are using Umbraco 10.5.0 and Deploy 10.2.3

pic01

pic01

AndyButland commented 1 year ago

Thanks, I've been able to replicate now. This issue occurs when using the legacy grid, and here the alt text/caption is set not on the media item but as part of the grid content (so the comment I made above about transferring of media isn't relevant).

Jay-1975 commented 1 year ago

So, based on the fact we are currently using the legacy grid there is no other way around this issue other than recreating all the legacy grids to block grid? I'll get back to you if I get anything back from our Ottawa team. Thanks Andy.

AndyButland commented 1 year ago

No, not at all - we still support the legacy grid with Deploy. I've been looking into it this morning and a fix will be available with the next patch release we make available for Deploy.

Jay-1975 commented 1 year ago

Thanks for looking into this Andy. I have let the Ottawa team know and see if they have any other questions regarding this issue. A question I have is when can we expect to see a patch release for Deploy?

AndyButland commented 1 year ago

It's not scheduled yet. We just released the last patch so this is the first item for the next one, but I'd expect in the next few weeks.

Jay-1975 commented 1 year ago

Sounds good! Thanks again for your help on this Andy. Unfortunately, we were in a spot where we'd have to recreate all of our Legacy grids to block grid and it would have been a lot of tedious work, especially as we are set to launch the Live site around September 12th. At least we know we will eventually get the functionality needed to move forward.

AndyButland commented 1 year ago

Understood. Yes, one day I imagine you'll have that migration task as it is the intention that block grid supersedes the existing grid... but certainly no need to do it for this issue.

Jay-1975 commented 1 year ago

Thanks Andy, I'll close the issue now and we will keep our eye out for the Deploy patch.

AndyButland commented 1 year ago

Thanks, but just to follow usual process, I'll keep it open until the related PR is reviewed and merged... then I'll close and label with the release numbers.

Jay-1975 commented 1 year ago

I see, I wasn't aware of the process.