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

Update R/shinymod snippet #469

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Update R/shinymod snippet #469

wants to merge 2 commits into from

Conversation

wurli
Copy link

@wurli wurli commented Jun 14, 2024

The previously used module syntax has been deprecated since 2020, see Mastering Shiny for details.

Thanks!

The previously used module syntax has been deprecated for quite some time
Copy link
Collaborator

@OkelleyDevelopment OkelleyDevelopment left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! Just a small change needed before we can land this into main. 😎

snippets/r.json Outdated Show resolved Hide resolved
@OkelleyDevelopment
Copy link
Collaborator

Any updates on this PR @wurli ?

@wurli
Copy link
Author

wurli commented Jul 9, 2024

Sorry for the late response! Just started a new job and a few things dropped off my radar. Thanks for the reminder and good catch! Just made the change.

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.

None yet

2 participants