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

Writer HTML: Fixed null string for Text Elements #2738

Merged
merged 1 commit into from
Feb 3, 2025
Merged

Writer HTML: Fixed null string for Text Elements #2738

merged 1 commit into from
Feb 3, 2025
+48 −9

Conversation

Progi1984
Copy link
Member

@Progi1984 Progi1984 commented Feb 3, 2025 •
edited
Loading

Description

Superseeded by #2629 by @armagedon007

Checklist:

  • My CI is 🟢
  • I have covered by unit tests my new code (check build/coverage for coverage report)
  • I have updated the documentation to describe the changes
  • I have updated the changelog

@Progi1984 Progi1984 added the HTML label Feb 3, 2025
@Progi1984 Progi1984 added this to the 1.4.0 milestone Feb 3, 2025
@Progi1984 Progi1984 mentioned this pull request Feb 3, 2025
Closed
4 tasks
@coveralls
Copy link

coveralls commented Feb 3, 2025 •
edited
Loading

Coverage Status

coverage: 96.749%. remained the same
when pulling d688dda on pr2629
into 2f270f2 on master.

@Progi1984 Progi1984 merged commit 1be7a80 into master Feb 3, 2025
59 checks passed
@Progi1984 Progi1984 deleted the pr2629 branch February 3, 2025 08:13
@Progi1984
Copy link
Member Author

Thanks @armagedon007 for your contribution 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
HTML
Milestone
1.4.0
Development

Successfully merging this pull request may close these issues.
3 participants
@Progi1984 @coveralls @armagedon007