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

UILexicon not loading into keyboard via expandLanguageDataset() #292

Closed
2 tasks done
andrewtavis opened this issue Apr 19, 2023 · 10 comments
Closed
2 tasks done

UILexicon not loading into keyboard via expandLanguageDataset() #292

andrewtavis opened this issue Apr 19, 2023 · 10 comments
Assignees
Labels
-priority- High priority bug Something isn't working help wanted Extra attention is needed

Comments

@andrewtavis
Copy link
Member

andrewtavis commented Apr 19, 2023

Terms

Behavior

The first bug in v2.2.0 is that it appears that expandLanguageDataset() isn't being called. What this means is that the user doesn't have access to all that was being added via this function call: specifically UILexicon names, the word Scribe so that it can trigger the easter egg 🐰🐣😢, and German compound prepositions.

The point in the app where this should be called is the following:

The issue is that as of now I cannot reproduce this on a simulator... A fix for this should definitely be included in a future release :)

Device type

iPhone and iPad

Versions

v2.2.0

@andrewtavis
Copy link
Member Author

andrewtavis commented Apr 20, 2023

Note that the following Scribe-Data issues will handle parts of this issue such that in the end the focus will just be the UILexicon not being included:

@andrewtavis
Copy link
Member Author

As the above specified Scribe-Data issues are done, the focus of this issue is now solely on the UILexicon not being present :)

@Taufi
Copy link
Contributor

Taufi commented Apr 25, 2024

I could take care of that. The problem seems to be that we are not allowed to write to files or databases on the app's main bundle. The simulator seems to allow it, but not the device.

@andrewtavis
Copy link
Member Author

Would be great to get some help with this, @Taufi! Thank you for offering :) Let us know if there's anything we can do to help 😊

@Taufi
Copy link
Contributor

Taufi commented May 3, 2024

Andrew, I was just about to make the pull request for the issue. Before that, I fetched the latest changes from upstream. This version ran into an error in Scripe>ParentTableCellModel from commit "b8cef23“. The init function does not work when setting the property hasNestedNavigation. The commit is from you, so the question is whether you can help?

@andrewtavis
Copy link
Member Author

Hey @Taufi 👋 It looks like I mistakenly removed self from self.hasNestedNavigation = hasNestedNavigation? Change should go in ParentTableCellModel.swift.

@andrewtavis
Copy link
Member Author

Do you want to add this as a minor fix to your PR?

@Taufi
Copy link
Contributor

Taufi commented May 3, 2024

Yes, with pleasure. Thanks for the tip.

@andrewtavis
Copy link
Member Author

Thanks for the fix, and sorry it was needed!

Taufi added a commit to Taufi/Scribe-iOS that referenced this issue May 3, 2024
@Taufi Taufi mentioned this issue May 3, 2024
1 task
andrewtavis added a commit that referenced this issue May 3, 2024
@andrewtavis
Copy link
Member Author

Closed via #420 🥳 I was seeing Appleseed as an autocomplete option across keyboards, and here's hoping we'll see it once v3.0 is live. Assuming we will 😊

Thanks @Taufi!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-priority- High priority bug Something isn't working help wanted Extra attention is needed
Projects
Archived in project
Development

No branches or pull requests

2 participants