devdocs icon indicating copy to clipboard operation
devdocs copied to clipboard

Incorrect versions in changelog of v1.1.14 of the quality patches

Open hostep opened this issue 3 years ago • 12 comments

Is there an existing issue for this?

  • [X] I have searched the existing issues

Which topic?

https://devdocs.magento.com/quality-patches/release-notes.html

What's wrong with the content?

It mentions Magento versions 1.3.0, 1.3.1 & 1.3.4, which never existed, so that seems to be a typo.

Screen Shot 2022-05-11 at 10 47 11

What changes do you propose?

Fix the version numbers

Anything else that can help to cover this?

No response

hostep avatar May 11 '22 08:05 hostep

Hi @hostep. Thank you for your report. To speed up processing of this issue, make sure that you provided sufficient information.

Add a comment to assign the issue: @magento I am working on this


m2-assistant[bot] avatar May 11 '22 08:05 m2-assistant[bot]

Under 1.1.13 there is another one:

MDVA-43451 (for Adobe Commerce and Magento Open Source >=1.3.2 <1.3.4)-Fixes the issue where the error “The store that was requested wasn’t found. Verify the store and try again.” appears while configuring a shared catalog for a specific website.

And under 1.0.21 another one:

MDVA-36424 (for Magento >=1.3.0 <=1.3.3-p1 || >=2.0.0 <2.2.0)-Fixes the issue where media images, attached to page builder elements, disappear when the content is being edited repeatedly, if the backend base URL is different from the storefront base URL.

Maybe there are more? I didn't search further...

hostep avatar May 11 '22 08:05 hostep

For the first two mentioned, it looks like those version ranges apply to the B2B product, and not the normal Commerce or Open Source products:

Screen Shot 2022-05-11 at 10 55 57

hostep avatar May 11 '22 08:05 hostep

@apoltoratskyi, please take a look and advise.

jeff-matthews avatar May 11 '22 13:05 jeff-matthews

@jeff-matthews: why did this issue get closed?

hostep avatar May 11 '22 19:05 hostep

@hostep, because the https://github.com/magento/devdocs/commit/c280c70528901af47db1b5b7e43a66fd4bfba0e5 commit fixed it and the PR that merged that commit automatically closed this issue. It looks like I need to force a deploy though, since the changes didn't affect file size and the web page wasn't updated. I just kicked off a deployment. It should be finished in about 10 minutes.

jeff-matthews avatar May 11 '22 19:05 jeff-matthews

Ah okay great! It's probably not being mentioned automatically over here since it was done in a private repository: https://github.com/magento-commerce/devdocs/

Any reason why this is not done over here on the public repo directly?

hostep avatar May 11 '22 19:05 hostep

Also, this commit doesn't seem to fix:

MDVA-36424 (for Magento >=1.3.0 <=1.3.3-p1 || >=2.0.0 <2.2.0)-Fixes the issue where media images, attached to page builder elements, disappear when the content is being edited repeatedly, if the backend base URL is different from the storefront base URL.

hostep avatar May 11 '22 19:05 hostep

Ah okay great! It's probably not being mentioned automatically over here since it was done in a private repository: https://github.com/magento-commerce/devdocs/

Any reason why this is not done over here on the public repo directly?

Yeah, you're right. All PRs are actually delivered via private repos in the magento-commerce org. That's why you see us adding the @magento import... comment on external PR contributions.

For internal PR contributions (those submitted by Adobe staff), it's more efficient to open the PR directly in the private repo so that we don't need to import it and put it through an unnecessary layer of processing.

jeff-matthews avatar May 11 '22 19:05 jeff-matthews

Also, this commit doesn't seem to fix:

MDVA-36424 (for Magento >=1.3.0 <=1.3.3-p1 || >=2.0.0 <2.2.0)-Fixes the issue where media images, attached to page builder elements, disappear when the content is being edited repeatedly, if the backend base URL is different from the storefront base URL.

That wasn't part of the original report, so we didn't address that. It looks like that's part of the 1.0.21 release notes.

jeff-matthews avatar May 11 '22 19:05 jeff-matthews

@jeff-matthews - I just checked this page : https://devdocs.magento.com/quality-patches/release-notes.html and all the version numbers have been corrected. Is there a reason why this issue is still open?

BarnyShergold avatar May 16 '22 10:05 BarnyShergold

@BarnyShergold, yes there is, please read ALL the comments in this thread 😉

hostep avatar May 16 '22 10:05 hostep

Looks like my last remark eventually got fixed in https://github.com/magento/devdocs/commit/3c7a6ad7df8b7b9cdac19e1f1e52218f24e212b1

Let's close this issue now finally.

hostep avatar Oct 10 '23 21:10 hostep