-
Notifications
You must be signed in to change notification settings - Fork 3
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
🚧 Feature - add multiple tracking properties #40
base: master
Are you sure you want to change the base?
🚧 Feature - add multiple tracking properties #40
Conversation
Requires an update of the Logic App definition (I think) to write positive tests that include multiple tracking properties, @mbraekman ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - Happy test like you suggested would be ideal indeed!
Currently the
Yes, we should update the logic app to support multiple tracked properties. Purpose should be:
I'll assign this to myself to create a proper Logic App to be used for this, unless anyone else wants to do this of course :) |
This was finished, right @mbraekman or did I misread this somewhere else? |
@tomkerkhove, not yet - a different logic app has been created to be used for the Update/Run-tests, but it didn't support multiple tracking properties yet. So, still in progress. |
No worries! |
Relates to #33