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

feat: add useRemoteAtServer to PutRequestOptions and (new) DeleteRequestOptions #1025

Merged
merged 6 commits into from
May 10, 2023

Conversation

gkc
Copy link
Contributor

@gkc gkc commented May 10, 2023

- What I did

  • feat: Add useRemoteAtServer to PutRequestOptions. When set, the update
    request will be sent directly to the remote atServer
  • feat: Introduce DeleteRequestOptions
    • Add new optional named parameter deleteRequestOptions to AtClient.delete
    • Add useRemoteAtServer to DeleteRequestOptions. When set, the delete
      request will be sent directly to the remote atServer
  • test: Added tests for the above
  • build: update pubspec.yaml and CHANGELOG.md
  • docs: Documentation for GetRequestOptions, PutRequestOptions, DeleteRequestOptions

- How to verify it
Tests should pass

gkc added 6 commits May 10, 2023 13:57
…pdate

  request will be sent directly to the remote atServer
- feat: Introduce DeleteRequestOptions
  - Add new optional named parameter `deleteRequestOptions` to AtClient.delete
  - Add `useRemoteAtServer` to DeleteRequestOptions. When set, the delete
    request will be sent directly to the remote atServer
- test: Add tests for the above
- build: update pubspec.yaml and CHANGELOG.md
- docs: Documentation for GetRequestOptions, PutRequestOptions, DeleteRequestOptions
… to the put and delete methods in AtClient spec
…eliminate this analysis warning which is due to the use of the `unawaited` function which was added in Dart 2.15.0 `warning - lib/src/service/notification_service_impl.dart:401:9 - This API is available since SDK 2.15.0, but constraints '>=2.12.0 <3.0.0' don't guarantee it. Try updating the SDK constraints. - sdk_version_since`
Copy link
Member

@nickelskevin nickelskevin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no philosophical issues

@gkc gkc merged commit d692998 into trunk May 10, 2023
6 checks passed
@gkc gkc deleted the gkc-put-request-option-to-force-use-remote-secondary branch May 10, 2023 17:43
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

Successfully merging this pull request may close these issues.

None yet

2 participants