acookin/trustcenter #818
acookin/trustcenter #818
Conversation
acookin
commented
Jun 8, 2025
acookin
commented
- trust center stuff
- stuff
- tc stuff
github-actions
bot
added
dbschema
graphqlschema
migrations
cli
httpserve
graphapi
coreclient
config
labels
| func (t TrustCenterSetting) Mixin() []ent.Mixin { | ||
| return mixinConfig{ | ||
| additionalMixins: []ent.Mixin{ | ||
| newOrgOwnedMixin(t), |
There was a problem hiding this comment.
I know this is still a draft, but thought I'd comment early on this - instead of having this as org owned, it should be "owned" by the trust center object. We have an annotation you can use, if you look at groups as an example, I think this should easily be able to work the same. And then you also don't need the edge to trust center settings on the organization object.
entfga.SettingsChecks("group"),
There was a problem hiding this comment.
You would set that ^ first, and then you can add this to your policy on a second run:
entfga.CheckEditAccess[*generated.TrustCenterSettingMutation](),
There was a problem hiding this comment.
On the first generate, there's a failure with the authcheck dereferencing a string, and I honestly do not understand why. Not sure if i need to generate from scratch, but not even sure how to do that because if i delete the generated directory(s), i get massive errors
There was a problem hiding this comment.
I retried this whole thing bc i was going around in circles, this seems to fail with different errors, so that's fun:
#822
acookin
force-pushed
the
acookin/trustcenter
branch
from
43573c5 to
15e1e77
Compare
