Skip to content
This repository was archived by the owner on May 13, 2024. It is now read-only.

chore: add trackjs #284

Merged
merged 2 commits into from
Mar 18, 2024
Merged

Conversation

markw-deriv
Copy link
Contributor

@markw-deriv markw-deriv commented Mar 13, 2024

Add TrackJS import to add error logging and tracking to the site.

TRACKJS_TOKEN will need to be added to the action environment variables. I don't have access to do this.

Copy link

@review-deriv review-deriv left a comment

Choose a reason for hiding this comment

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

PR description is too short. You can use this format:

# Description
This is a summary of the changes. This should describe why do we need this change.
Whys and Hows come here.

# Cards
* https://clickup.com/12345
* https://clickup.com/12346

Copy link

vercel bot commented Mar 13, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-api-docs ✅ Ready (Inspect) Visit Preview Mar 15, 2024 8:31am

Copy link

gitguardian bot commented Mar 13, 2024

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Our GitHub checks need improvements? Share your feedbacks!

@coveralls
Copy link

coveralls commented Mar 13, 2024

Pull Request Test Coverage Report for Build 8293300462

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 97.215%

Totals Coverage Status
Change from base Build 8200071149: 0.0%
Covered Lines: 1698
Relevant Lines: 1708

💛 - Coveralls

Co-authored-by: Jim Daniels Wasswa <104334373+jim-deriv@users.noreply.github.com>
@sandeep-deriv sandeep-deriv merged commit a3e2ef1 into binary-com:master Mar 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants