-
Notifications
You must be signed in to change notification settings - Fork 888
feat: add deployment_id
to the ui and licenses
#13096
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
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'll let @johnstcn reviewing the go code - but the UI LGTM.
site/src/modules/dashboard/Navbar/UserDropdown/UserDropdownContent.tsx
Outdated
Show resolved
Hide resolved
site/src/modules/dashboard/Navbar/UserDropdown/UserDropdownContent.tsx
Outdated
Show resolved
Hide resolved
@@ -17,6 +17,9 @@ | |||
}, | |||
{ | |||
"pattern": "tailscale.com" | |||
}, | |||
{ | |||
"pattern": "wireguard.com" |
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.
sweet ❤️
@kylecarbs it is not showing up on dev.coder.com although the version v2.10.0-devel+1bda8a085 have been deployed successfully. |
Hmm, I'll check why that is. Odd |
@matifali fixed! |
Allows customers to send us the deployment ID easily. I put this in the user dropdown because it's so global. If users find this confusing, we can happily move it!

Soon, licenses can be generated with specific deployment IDs to match against. This is to prevent customers from having a bunch of Coder deployments with the same license.