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

#8979 Support attr text:continue-numbering on ODT Reader #8998

Merged
merged 9 commits into from
Nov 7, 2023

Conversation

StephanMeijer
Copy link
Contributor

@StephanMeijer StephanMeijer commented Aug 13, 2023

Please keep the following in mind: I am a beginner in Haskell. So in case I wrote garbage, please don't judge me too harshly. Please also check thoroughly before merging. Thanks!

See #8979


  • Update test .odt to be from LibreOffice not Google Docs, so .native-output isn't cluttered with BlockQuote.
  • Fix typo @ZwaarContrast caught.

@StephanMeijer
Copy link
Contributor Author

@jgm I refactored the code to make the functionality more clear.

@StephanMeijer StephanMeijer changed the title #8979 Support attr text:continue-numbering on ODT Reader #8979 Support attr text:continue-numbering on ODT Reader Aug 14, 2023
@StephanMeijer StephanMeijer force-pushed the feature/8979-continue-numbering branch from 47f1915 to e9bf624 Compare August 17, 2023 13:56
@StephanMeijer
Copy link
Contributor Author

@jgm Updated PR with cleaner example for test. Would you be willing to review once again? 🙏

Copy link
Owner

@jgm jgm left a comment

Choose a reason for hiding this comment

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

This looks good; I've just made a couple small suggestions.

@StephanMeijer
Copy link
Contributor Author

@jgm Feedback taken into account and changed things appropriately. I didn't test locally so I will test on CI/CD.

@jgm jgm merged commit 56f7f11 into jgm:main Nov 7, 2023
9 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants