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

feat: operated by text #655

Merged
merged 2 commits into from
Aug 21, 2024
Merged

feat: operated by text #655

merged 2 commits into from
Aug 21, 2024

Conversation

yogurtandjam
Copy link
Contributor

related to https://linear.app/dydx/issue/OTE-721/add-operated-by-text-to-dydxtrade-website

adding text to make it clear that dydx trading does not operate any instance of the dydx.trade website or other deployments of the code

Copy link

linear bot commented Aug 21, 2024

Copy link

vercel bot commented Aug 21, 2024

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

Name Status Preview Comments Updated (UTC)
v4-localization ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 21, 2024 3:58pm

@yogurtandjam yogurtandjam merged commit 42b9db0 into main Aug 21, 2024
5 checks passed
@yogurtandjam yogurtandjam deleted the jerms/operated-by-text branch August 21, 2024 16:05
@@ -872,6 +872,8 @@
"SIGN_IN_WITH_EMAIL": "Sign in with email",
"SIGN_IN_WITH_EMAIL_SUBTITLE": "A simple way to sign in, powered by {MAGIC}",
"SKIP_FOR_NOW": "Skip for now",
"SITE_OPERATED_BY_SHORT": "This site is operated by {NAME_OF_DEPLOYER}, utilizing software open sourced by dYdX Trading Inc.  {LEARN_MORE_LINK}.",
"SITE_OPERATED_BY_LONG": "This site is operated by {NAME_OF_DEPLOYER} subject to [deployer T&Cs.], utilizing certain software open sourced by third party dYdX Trading Inc. subject to its Terms of Use, which prohibit use of the software by persons based in the United States or other restricted jurisdictions. The dYdX blockchain is powered by a community of independent validators. {LEARN_MORE_LINK}.",
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: I think you have a double space before Learn More

@@ -872,6 +872,8 @@
"SIGN_IN_WITH_EMAIL": "Sign in with email",
"SIGN_IN_WITH_EMAIL_SUBTITLE": "A simple way to sign in, powered by {MAGIC}",
"SKIP_FOR_NOW": "Skip for now",
"SITE_OPERATED_BY_SHORT": "This site is operated by {NAME_OF_DEPLOYER}, utilizing software open sourced by dYdX Trading Inc.  {LEARN_MORE_LINK}.",
Copy link
Contributor

Choose a reason for hiding this comment

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

also double space here; I think I get this when I copy paste from notion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants