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.

Sign up for GitHub

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

Jump to bottom

feat: add uuids to subscriptions #37

Merged

im-adithya merged 1 commit into main from task-uuid

Apr 2, 2024

Member

im-adithya commented Apr 2, 2024

Adds uuid column to the subscriptions table
Subscriptions are now stopped using UUID and not id (making it difficult to delete others' channels)


          feat: add uuids to subscriptions

6c2460b

im-adithya merged commit 5a3d531 into main

1 check passed

bumi deleted the task-uuid branch

April 2, 2024 11:24

bumi reviewed

View reviewed changes

internal/nostr/models.go

@@ @@ -34,6 +34,7 @@ type Subscription struct { @@
               	Search        string
               	CreatedAt     time.Time
               	UpdatedAt     time.Time
+              	Uuid          string        `gorm:"type:uuid;default:gen_random_uuid()"`

Contributor

bumi Apr 2, 2024

does this have an index then automatically?

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

Labels

None yet