Adding support for schemas #23
Closed
Akamatsu21
started this conversation in
Show and tell
Replies: 2 comments 2 replies
-
|
Hi @Akamatsu21 - for sure @omer9564 will be able to review this once you have a PR, might take a little while to do so, but we can definitely make sure to cover it (CC: @obsd ) |
Beta Was this translation helpful? Give feedback.
1 reply
-
|
@Akamatsu21 fixed it in the PR, it was merged and released as cedar-agent version |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hello, as per the discussion in this thread, myself and @tisauro have decided to make an attempt at implementing the schema support for cedar-agent. I have made a new branch for this which you can find in this repo. This is a rough plan I've been following with this:
I've not created a pull request for now as the feature isn't finished yet (although I am happy to make one if it makes it easier to browse the changes?). I wanted to ask if one of the main contributors to this project would be able to spare some time and have a look through what I've done so far, to confirm that this is in line with your expectations and there aren't any glaring mistakes? I would consider myself intermediate at Rust, so I'd be very happy to learn if anything could have been done better.
Beta Was this translation helpful? Give feedback.
All reactions