-
Notifications
You must be signed in to change notification settings - Fork 0
add hotjar and feeback google form btn #5
add hotjar and feeback google form btn #5
Conversation
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.
With no env-vars, I'm guessing this is exposing our hotjar ID/key (or similar)?
I doubt anyone will submit fake hotjar data, so I'm happy with doing it in the short-term. Probably good to move it at some point.
|
Good shout I should fix this |
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.
Does this google form only allow responses from .hackney.gov.uk emails?
(LGTM either way!)
@@ -47,6 +47,7 @@ jobs: | |||
export AIRTABLE_TABLE_NAMES=${AIRTABLE_TABLE_NAMES_INTEGRATION} | |||
export ADDRESSES_API_URL=${ADDRESSES_API_URL} | |||
export ADDRESSES_API_KEY=${ADDRESSES_API_KEY} | |||
export HOTJAR_ID=${HOTJAR_ID} |
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.
I'm still getting my head around the split configuration - I see no reason why this shouldn't work, but if it doesn't, that'd be my first suspicion!
What
![image](https://user-images.githubusercontent.com/63246336/90233126-8b3e0b00-de15-11ea-85d3-a21964497aa0.png)
Adds a Submit feedback btn to a google form and a HotJar tracking script
Why
For user Testing
Anything else?