-
Notifications
You must be signed in to change notification settings - Fork 4
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
Setup fullstory recording #34
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.
I had a question and a suggestion for the code, but mostly looks good. Have you tested this locally or on staging to see if it works?
Thanks @dpca for reviewing, let me try getting local working on uploading data to fullstory before we merge this. |
Block these elements to be fullstory recorded according to Product requirement
@dpca I tested locally and it works as expected with these additional commits. I also added Add fs-block to genome name, fragment name, sequence per Product's request to prevent possible leak of proprietary information. |
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.
Looks good!
https://fullstory.com/