Skip to content

Conversation

basant20415
Copy link
Contributor

here is the output of the function
listMessageLogs

if you need any other modification ,please let me know

@pooranjoyb
Copy link
Owner

please resolve conflicts

@basant20415
Copy link
Contributor Author

are there any problems with this pull request?

return response;
} else {
throw AppwriteException("Error listing message logs. Status code: " +

Copy link
Owner

Choose a reason for hiding this comment

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

where is the } ?

Copy link
Contributor Author

@basant20415 basant20415 Aug 6, 2025

Choose a reason for hiding this comment

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

it might be deleted when i tried to resolve conflicts ,should i create another pull request for this function or how can i add it?

Copy link
Owner

Choose a reason for hiding this comment

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

no need to create another pull request, do your changes in this branch and push into the same branch itself

Copy link
Contributor Author

Choose a reason for hiding this comment

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

this closing brace is already added in my code ,i think it's deleted while i'm trying to resolve the conflicts,i'll try to make any small changes in the function and push it again with the brace

Copy link
Owner

Choose a reason for hiding this comment

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

ok

@pooranjoyb pooranjoyb changed the title list message logs function feat: listMessageLogs [messaging] Aug 6, 2025
@pooranjoyb pooranjoyb added mild-mayhem purple litmus paper (pH 7) gssoc25 Issue under Girlscript Summer Of Code 2025 level 2 GSSOC level 2 labels Aug 6, 2025
@pooranjoyb pooranjoyb moved this to In Progress in C++ SDK Appwrite Aug 6, 2025
@basant20415
Copy link
Contributor Author

Does it need any other modifications, or is it okay?

@basant20415
Copy link
Contributor Author

I hope everything is still fine and I didn’t mess anything up

@pooranjoyb
Copy link
Owner

I hope everything is still fine and I didn’t mess anything up

okay will verify and test once.

@pooranjoyb
Copy link
Owner

I hope everything is still fine and I didn’t mess anything up

When i test it with a message, I cannot the logs of the messages, can you show me the screenshots of appwrite console and ids you're using to fetch the logs? @basant20415

@basant20415
Copy link
Contributor Author

basant20415 commented Aug 8, 2025

image image this is the message i created
@pooranjoyb
Copy link
Owner

okay testing.

@basant20415
Copy link
Contributor Author

is everything okay in the function?

@pooranjoyb
Copy link
Owner

is everything okay in the function?

yes thank you.

@pooranjoyb pooranjoyb merged commit 7ffd420 into pooranjoyb:develop Aug 11, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in C++ SDK Appwrite Aug 11, 2025
@pooranjoyb pooranjoyb linked an issue Aug 11, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gssoc25 Issue under Girlscript Summer Of Code 2025 level 2 GSSOC level 2 mild-mayhem purple litmus paper (pH 7)

2 participants