This repository was archived by the owner on Sep 30, 2024. It is now read-only.
Commit e9a31ea
authored
feat/enterpriseportal: use UUID type for subscription ID (#63452)
We were aligned on using non-prefixed UUIDs internally, only prefixing
them for external consumption. This lets us use the native UUID type.
Part of https://linear.app/sourcegraph/issue/CORE-155
## Test plan
CI, `sg run enterprise-portal`
`select * from enterprise_portal_subscriptions;` on dev shows we are
using the non-prefixed UUDs only. Manual migration, since it doesn't
seem like GORM can do this for you:
```
ALTER TABLE enterprise_portal_subscriptions ALTER COLUMN id TYPE uuid USING id::uuid;
```
Checked that data is intact.1 parent 6c861d1 commit e9a31ea
File tree
2 files changed
+8
-4
lines changed- cmd/enterprise-portal/internal/database/subscriptions
2 files changed
+8
-4
lines changedLines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
13 | 13 | | |
14 | 14 | | |
15 | 15 | | |
16 | | - | |
17 | | - | |
| 16 | + | |
| 17 | + | |
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
| |||
Lines changed: 6 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
67 | 67 | | |
68 | 68 | | |
69 | 69 | | |
70 | | - | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
71 | 73 | | |
72 | 74 | | |
73 | 75 | | |
| |||
83 | 85 | | |
84 | 86 | | |
85 | 87 | | |
86 | | - | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
87 | 91 | | |
88 | 92 | | |
89 | 93 | | |
| |||
0 commit comments