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
Fix: Groups support for Secret Approvals and Access Requests #1773
base: daniel/request-access
Are you sure you want to change the base?
Conversation
t.foreign("approverUserId").references("id").inTable(TableName.Users).onDelete("CASCADE"); | ||
}); | ||
|
||
await knex.schema.alterTable(TableName.SecretApprovalRequest, (t) => { |
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 was testing identity for secret replication and issue is then i would need to add all these fields again . May be instead we do it like permission there is non nullable - `type"which says whether its user or identity and keep rest as generic like before
Then do a conditional join
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.
Kindly make the migration file idempotent
backend/src/ee/services/access-approval-request/access-approval-request-dal.ts
Outdated
Show resolved
Hide resolved
backend/src/ee/services/access-approval-request/access-approval-request-dal.ts
Outdated
Show resolved
Hide resolved
backend/src/db/migrations/20240429172301_access_approval_requests.ts
Outdated
Show resolved
Hide resolved
…sical into daniel/fix-db-ref
06871fc
to
9a585ad
Compare
No description provided.