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

Refactor public interfaces #54

Open
elf-pavlik opened this issue Apr 17, 2023 · 0 comments
Open

Refactor public interfaces #54

elf-pavlik opened this issue Apr 17, 2023 · 0 comments
Assignees
Labels
application Top level module for applications authorization-agent Top level module for authorization agents data-model Basic interfaces for data model

Comments

@elf-pavlik
Copy link
Collaborator

elf-pavlik commented Apr 17, 2023

Given all the functionality in place and experience with implementing both authorizations agent and demo applications, we can make quite a lot of improvement.

Suggested in #51 (comment)

@angelaraya: Also, while I like compressing parameters into option objects in this case it obfuscates the inputs needed by the function. I'd rather have as many parameters as needed here, specially because the type name ShareDataInstanceStructure does not convey at all what it actually is.


@elf-pavlik elf-pavlik self-assigned this Apr 17, 2023
@elf-pavlik elf-pavlik added authorization-agent Top level module for authorization agents application Top level module for applications data-model Basic interfaces for data model labels Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application Top level module for applications authorization-agent Top level module for authorization agents data-model Basic interfaces for data model
Projects
None yet
Development

No branches or pull requests

1 participant