Skip to content
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

Sketch A: Allow sync formula to return ACLs #3039

Closed
wants to merge 5 commits into from

Conversation

patrick-codaio
Copy link
Contributor

No description provided.

@chris-codaio
Copy link
Contributor

I suppose this boils down to a special short-cut case of something like the ServiceNow pack where the schema for KnowledgeArticle contains the ACLs inline. Today, we just take in the row and spit out the permissions through a synchronous transform - no APIs involved. Not the end of the world if we have to implement the getPermissions handler for this case, though, like we do already. I think I agree with you and others that your Sketch B proposal has more utility.

Copy link

github-actions bot commented Sep 4, 2024

This pull request has been inactive for 7 days: labeled as stale. To avoid auto-closure in 7 days, please do one of the following: Add keep-active label, comment on PR, push new commit on PR.

@github-actions github-actions bot added the stale label Sep 4, 2024
Copy link

This stale PR has been inactive for 14 days; Closing PR.

@github-actions github-actions bot closed this Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants