Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Documentation for BC Breaks and changes in 8.0 #1427

Merged
merged 68 commits into from
Aug 16, 2022

Conversation

kpodemski
Copy link
Contributor

@kpodemski kpodemski commented Jul 28, 2022

PRs:
PrestaShop/PrestaShop#12428
PrestaShop/PrestaShop#24878
PrestaShop/PrestaShop#25066
PrestaShop/PrestaShop#25914
PrestaShop/PrestaShop#25872
PrestaShop/PrestaShop#25956
PrestaShop/PrestaShop#26506
PrestaShop/PrestaShop#26609
PrestaShop/PrestaShop#26716
PrestaShop/PrestaShop#27364
PrestaShop/PrestaShop#27756
PrestaShop/PrestaShop#27657
PrestaShop/PrestaShop#27469
PrestaShop/PrestaShop#27442
PrestaShop/PrestaShop#27914
PrestaShop/PrestaShop#27751
PrestaShop/PrestaShop#28086
PrestaShop/PrestaShop#27851
PrestaShop/PrestaShop#27159
PrestaShop/PrestaShop#27439
PrestaShop/PrestaShop#28191
PrestaShop/PrestaShop#28192
PrestaShop/PrestaShop#28193
PrestaShop/PrestaShop#28190
PrestaShop/PrestaShop#28132
PrestaShop/PrestaShop#28194
PrestaShop/PrestaShop#28198
PrestaShop/PrestaShop#28197
PrestaShop/PrestaShop#28195
PrestaShop/PrestaShop#28545
PrestaShop/PrestaShop#28578
PrestaShop/PrestaShop#28461
PrestaShop/PrestaShop#28196
PrestaShop/PrestaShop#28177
PrestaShop/PrestaShop#28021
PrestaShop/PrestaShop#28020
PrestaShop/PrestaShop#28010
PrestaShop/PrestaShop#27961
PrestaShop/PrestaShop#27959
PrestaShop/PrestaShop#27958
PrestaShop/PrestaShop#27957
PrestaShop/PrestaShop#28009
PrestaShop/PrestaShop#27956

PrestaShop/PrestaShop#28163
PrestaShop/PrestaShop#27955
PrestaShop/PrestaShop#28068
PrestaShop/PrestaShop#27859
PrestaShop/PrestaShop#27892
PrestaShop/PrestaShop#27626
PrestaShop/PrestaShop#27723
PrestaShop/PrestaShop#28445
PrestaShop/PrestaShop#28277
PrestaShop/PrestaShop#27632
PrestaShop/PrestaShop#29054
PrestaShop/PrestaShop#27575
PrestaShop/PrestaShop#28504
PrestaShop/PrestaShop#27755
PrestaShop/PrestaShop#28402
PrestaShop/PrestaShop#26048 - example to be documented
PrestaShop/PrestaShop#28127 - example to be documented
PrestaShop/PrestaShop#27611

Questions Answers
Branch? 8.x
Description? documentation updates
Fixed ticket? Fixes #{issue number here} if there is a related issue

@github-actions github-actions bot added the 8.x label Jul 28, 2022
@kpodemski kpodemski marked this pull request as draft July 28, 2022 12:52
@kpodemski kpodemski changed the title Documentation for BC Breaks and changes Documentation for BC Breaks and changes in 8.0 Jul 28, 2022
@kpodemski kpodemski removed the request for review from eternoendless July 28, 2022 15:15
@kpodemski kpodemski marked this pull request as ready for review August 4, 2022 13:00
@ghost
Copy link

ghost commented Aug 4, 2022

oh very nice job; for this PR i update the doc : #1434

@kpodemski
Copy link
Contributor Author

Thanks, @okom3pom

We'll have to review all the types as we have some to add; I know about one added recently to the order page. I'll create an Issue.

modules/core-updates/8.0.md Outdated Show resolved Hide resolved
modules/core-updates/8.0.md Outdated Show resolved Hide resolved
## API Changes

### Changes

---
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Putting these separators will break the bullet points markup
we're going to get this

<ul><li>Item</li></ul>
<hr>
<ul><li>Second item</li></ul>
<hr>
<!-- ...and so on -->

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd keep it like this. It looks much better and more readable.

kpodemski and others added 2 commits August 11, 2022 14:07
Co-authored-by: Pablo Borowicz <pablo.borowicz@prestashop.com>
Copy link
Contributor

@matks matks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work and careful exploration. Thank you for doing all this @kpodemski

@kpodemski kpodemski merged commit 2d7d551 into PrestaShop:8.x Aug 16, 2022
modules/core-updates/8.0.md Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants