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 snowplow_mobile_delete_from_manifest macro #54

Closed
emielver opened this issue Jan 14, 2022 · 0 comments
Closed

Add snowplow_mobile_delete_from_manifest macro #54

emielver opened this issue Jan 14, 2022 · 0 comments
Assignees

Comments

@emielver
Copy link
Contributor

Describe the feature

We already have the snowplow_web_delete_from_manifest macro in snowplow_delete_from_manifest. We should make the equivalent macro for the snowplow-mobile package to make the API less cumbersome for the user

Describe alternatives you've considered

We can alternatively call the more generic snowplow_delete_from_manifest function with two arguments as follows:
{{ snowplow_utils.snowplow_delete_from_manifest(var('models_to_remove', []), ref('snowplow_mobile_incremental_manifest')) }}
however this is more cumbersome

Who will this benefit?

This will benefit all future users of the snowplow-mobile package that is currently in development

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

No branches or pull requests

1 participant