Skip to content

Feature/fix send #198

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.

Sign up for GitHub

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

Jump to bottom
Merged
nsoblath merged 38 commits into from
May 19, 2025
Merged

Feature/fix send #198

nsoblath merged 38 commits into from
May 19, 2025

Conversation

@nsoblath
Copy link
Member

@nsoblath nsoblath commented Apr 26, 2025

update and add/remove sections as needed

New Features

  • Introduced RequestHandler and RequestSender

Fixes

  • Fixed service send-message binding
  • Added test_service.py
  • Other random fixes

Testing

  • Added test_service.py with send-message testing

pkolbeck and others added 30 commits February 14, 2025 17:49
@pkolbeck
Added RequestSender class that should handle sending dl messages. Mod…
f9a86d9 
…ified Service to include this class.
@pkolbeck
Added request_sender class. Tested with service. Not tested with Inte…
63d0d55 
…rface yet.
@pkolbeck
Removed weird encapsulation fo response in do_get_request.
7907603
@pkolbeck
Changed do_get_request to handle returning dictionaries
a3c2ca2
@pkolbeck
Changed result_to_scarab_payload to handle dictionaries and any.
e344704
@pkolbeck
Adjusted do_get_request and result_to_scarab_payload to handle dicts …
e19cbb0 
…and an.
@pkolbeck
Merge branch 'bugfix/do_get_dicts' into feature/getsetcmd-mixin
5f19450 
Adding necessary functionality to get commands.
@pkolbeck
Added RequestReceiver class as a mixin for endpoint and service. Get …
4d595e4 
…returns with 'values' field without a specifier.
@pkolbeck
updated versions.
bb17798
@pkolbeck
undo
d54beca
@nsoblath
Add UUID setting from string test
3cfb40c
@pkolbeck
_check_lockout_key handles uuid objs and strs
2ca25d2
@pkolbeck
Implemented changes from pull request comments.
5cdfe03
@nsoblath
Add bindings to service::send() and comments on how send bindings are…
c28bde1 
… setup.
@nsoblath
Undoing the breaking changes I made in the previous commit to the _se…
705e32c 
…rvice trampoline class.
@nsoblath
Simplified on_get/set/cmd_request() function calls and added some doc…
1310ae4 
…umentation
@nsoblath
Fixing on_[op]_request() functions to correctly get trampolined calls…
1488b1b 
… from the C++ base class
@nsoblath
Improving tests in test_endpoint.py that check for raising of exceptions
e9326be
@nsoblath
Merge commit '51b4ec9d303b3ac54512e3028b96bf7c6887864c' into feature/…
49d46f8 
…getsetcmd-mixin
@nsoblath
Change class name to avoid confusion with dl-cpp classes: RequestRece…
8a29b91 
…iver --> RequestHandler
@nsoblath
Fixed a couple more names
d6f1c7e
@nsoblath
Added classes to __init__.py
ff8e495
@nsoblath
Merge branch 'develop' into feature/getsetcmd-mixin
f0fd43a
@nsoblath
Merge branch 'feature/getsetcmd-mixin' into develop
1096230
@nsoblath
Fixing deprecated utc timezone use
f3d492d
@nsoblath
Removing unnecessary imports
a07dd2b
@nsoblath
Fixing Service send-mesage API
98d1b6f
@nsoblath
Fixing service binding
a0315a7
@nsoblath
Add send_error_message binding
bf2d05a
@nsoblath
Removing unnecessary line
93610f4
@nsoblath nsoblath merged commit 8570a5d into develop May 19, 2025
3 checks passed
@nsoblath nsoblath deleted the feature/fix_send branch May 19, 2025 17:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nsoblath @pkolbeck