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

Remove version check from ipuz reader. #173

Merged
merged 1 commit into from
Dec 23, 2021
Merged

Conversation

jpd236
Copy link
Contributor

@jpd236 jpd236 commented Dec 17, 2021

v2 is generally a safe expansion on top of v1 - ipuz.org notes that some
fields have been deprecated, though the only explicit mention of
"deprecated" on the v2 doc relates to a field in Sudoku puzzles. There
may be some additional features that aren't yet supported, but these can
always be added later.

See #172

v2 is generally a safe expansion on top of v1 - ipuz.org notes that some
fields have been deprecated, though the only explicit mention of
"deprecated" on the v2 doc relates to a field in Sudoku puzzles. There
may be some additional features that aren't yet supported, but these can
always be added later.

See mrichards42#172
@mrichards42
Copy link
Owner

@crosswordnexus I thought I saw a comment about a v2 ipuz that xword can't load as v1?

@crosswordnexus
Copy link

@crosswordnexus I thought I saw a comment about a v2 ipuz that xword can't load as v1?

I was wrong about this. It's a v2 iPuz file that doesn't load in Xword for unrelated reasons. I can make an example file and open a second issue.

@mrichards42
Copy link
Owner

Got it, thanks!

@mrichards42 mrichards42 merged commit 584f78d into mrichards42:master Dec 23, 2021
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.

3 participants