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

at_c: verbbuilders in packages/atcommons #165

Open
26 tasks
Tracked by #39
JeremyTubongbanua opened this issue Mar 19, 2024 · 1 comment
Open
26 tasks
Tracked by #39

at_c: verbbuilders in packages/atcommons #165

JeremyTubongbanua opened this issue Mar 19, 2024 · 1 comment
Assignees

Comments

@JeremyTubongbanua
Copy link
Member

JeremyTubongbanua commented Mar 19, 2024

Implement VerbBuilders, just like how we have in our at_java and Dart at_client_sdk SDKs.

It is a common pattern to do atProtocol operations like update:<..> and llookup:<..>. seeing that this is a common pattern, current CRUD and notification operations should be refactored and utilize verb builders.

I am justifying this as "P3 - Nice to have" because it isn't core to the SDK's functionality

This should go in packages/atcommons to be utilized as a dependency by atclient and atauth

Tasks

Function signature would look something like this:

int atclient_verb_builders_build_put(char **output, const atclient_atkey *atkey, const char *value);
int atclient_verb_builders_build_scan(char **output, const char *regex, const bool show_hidden);
// ...
@JeremyTubongbanua
Copy link
Member Author

Backlogging since this is unimportant for C Daemon

@JeremyTubongbanua JeremyTubongbanua changed the title at_c: verbbuilders at_c: verbbuilders in packages/atcommons Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants