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.

Sign up for GitHub

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

Jump to bottom

[API] Fix formating for list in invoice cookbook #12888

Merged
merged 1 commit into from Aug 13, 2021
Merged

[API] Fix formating for list in invoice cookbook #12888

merged 1 commit into from Aug 13, 2021

Conversation

arti0090
Copy link
Contributor

Q A
Branch? master
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Related tickets fixes for #12879
License MIT

@arti0090 arti0090 requested a review from a team as a code owner August 13, 2021 08:44
@probot-autolabeler probot-autolabeler bot added the Documentation Documentation related issues and PRs - requests, fixes, proposals. label Aug 13, 2021
@Tomanhez
Copy link
Contributor

Tomanhez commented Aug 13, 2021 •
edited

Looks good but build is red

@Tomanhez Tomanhez merged commit 90ed451 into Sylius:master Aug 13, 2021
@Tomanhez
Copy link
Contributor

Thanks, @arti0090! 🥇

@arti0090 arti0090 mentioned this pull request Aug 13, 2021
Merged
Tomanhez added a commit that referenced this pull request Aug 13, 2021
@Tomanhez
bug #12890 Fix to ordered list (arti0090)
02961dd 
This PR was merged into the 1.11-dev branch.

Discussion
----------

Now the list looks good 🎉 Continues #12888


![image](https://user-images.githubusercontent.com/35863747/129346596-814d3162-1e02-4422-976e-c000e0cfba01.png)


Commits
-------

68ab620 Fix to ordered list
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Tomanhez Tomanhez Tomanhez approved these changes

Assignees
No one assigned
Labels
Documentation Documentation related issues and PRs - requests, fixes, proposals.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@arti0090 @Tomanhez