Skip to content

update to README.md#99

Open
mdbench wants to merge 2 commits intoSvenTiigi:mainfrom
mdbench:main
Open

update to README.md#99
mdbench wants to merge 2 commits intoSvenTiigi:mainfrom
mdbench:main

Conversation

@mdbench
Copy link
Copy Markdown

@mdbench mdbench commented Feb 4, 2025

This update will make it easier for users of your package to understand how to implement it manually. I had to thoroughly examine your source code to arrive at this basic syntax, including understanding your bindings, as the syntax was missing from the documentation.

Comment thread README.md
foregroundColor: .white,
hapticFeedback: .notification(.success),
onDismiss: {
UserDefaults.standard.set(false, forKey: "hasShowWhatsNew")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

There looks to be a typo in this key

Suggested change
UserDefaults.standard.set(false, forKey: "hasShowWhatsNew")
UserDefaults.standard.set(false, forKey: "hasShownWhatsNew")

Comment thread README.md Outdated
Co-authored-by: Andrew Elliott <26266444+andrewse02@users.noreply.github.com>
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