Skip to content

feat: add createSms API to Messaging service with example - #105

Open
SaidSafwan wants to merge 1 commit into
pooranjoyb:developfrom
SaidSafwan:feature/add-create-sms-api
Open

SaidSafwan wants to merge 1 commit into
pooranjoyb:developfrom
SaidSafwan:feature/add-create-sms-api

Conversation

@SaidSafwan

Copy link
Copy Markdown

Added support for sending SMS messages via Appwrite Messaging API.

Implemented new function createSms() in Messaging service.

Added usage example (examples/messaging/messages/createSms.cpp).

Updated CMakeLists.txt to build the example.

@pooranjoyb pooranjoyb left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Please show response from the api

Comment thread CMakeLists.txt
Comment on lines +52 to +53
add_executable(example-messaging-create-sms examples/messaging/messages/createSms.cpp)
target_link_libraries(example-messaging-create-sms AppwriteSDK) No newline at end of file

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

No changes to be made in this file

#include "enums/HttpStatus.hpp"
#include "exceptions/AppwriteException.hpp"
#include <string>
#include <vector>

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Why is vector imported ?

@sristy17

Copy link
Copy Markdown
Collaborator

any updates?

@pooranjoyb pooranjoyb added the commitment-issues basically stale PR label Sep 6, 2025

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

commitment-issues basically stale PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants