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

Add apple typos #38

Merged
merged 3 commits into from
Dec 18, 2024
Merged

Add apple typos #38

merged 3 commits into from
Dec 18, 2024

Conversation

Veeit
Copy link

@Veeit Veeit commented Dec 18, 2024

Should notify users who misspell iPhone or iPad.

@pascalfriedrich
Copy link
Contributor

pascalfriedrich commented Dec 18, 2024

Although your solution is not wrong, maybe we should add a more generic one? For example creating a dictionary with a list of wrong words and the corresponding correct word. With that, the code would be less repetitive and easier to extend.

Copy link
Contributor

@pascalfriedrich pascalfriedrich left a comment

Choose a reason for hiding this comment

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

Good job!

@pascalfriedrich pascalfriedrich merged commit 6544549 into swiftde:main Dec 18, 2024
1 check 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.

2 participants