Skip to content

Conversation

patrickkulling
Copy link
Contributor

@patrickkulling patrickkulling commented Jan 24, 2023

###Description

Resolves #579

@kakubin here is my first fix. Please have a look - happy for feedback.

@patrickkulling
Copy link
Contributor Author

@kakubin can you prioritize this PR as well?

Copy link
Contributor

@kakubin kakubin left a comment

Choose a reason for hiding this comment

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

Everything is fine except CHANGELOG.md

CHANGELOG.md Outdated
Comment on lines 2 to 3
- Fix gsub! usage for open office documents on a frozen string [580](https://github.com/roo-rb/roo/pull/580)
- Add support for boolean values in open office files that were generated via Google Sheets [579](https://github.com/roo-rb/roo/pull/579)
Copy link
Contributor

Choose a reason for hiding this comment

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

This new line should be end with link #580

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Let me fix this up. Must have been mixed up in the merge

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@kakubin Have another look now - the other line in the changelog was wrong as well. Both were using the issues numbers and not the PR numbers.

@patrickkulling patrickkulling merged commit 13677da into roo-rb:master Feb 7, 2023
@patrickkulling patrickkulling deleted the fix/open-office-boolean-google-sheets branch February 7, 2023 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for boolean values in open office files exported via Google Sheets

2 participants