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

teal vignettes #695

Closed
wants to merge 10 commits into from
Closed

teal vignettes #695

wants to merge 10 commits into from

Conversation

nikolas-burkoff
Copy link
Contributor

@nikolas-burkoff nikolas-burkoff commented Jul 13, 2022

Closes #682

Blocked: should be merged once insightsengineering/teal.code#49 has been merged in

  • We should also add an vignette showing using the filter_panel_api in your module (at the moment I just put see "teal.slice") - see PR comment inline - similar to the adding reporter to your custom module

function(id, datasets, ...) {
function(id,
data, # optional, use if you want your module to access the application data
filter_panel_api, # optional, use if you want access to the filter panel from your module see teal.slice
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This should say see "using the filter panel api in your modules" and that vignette should be created

@nikolas-burkoff
Copy link
Contributor Author

Closed in favour of #721

@cicdguy cicdguy deleted the 682_vignette@main branch September 24, 2022 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update vignette
1 participant