This repository has been archived by the owner on Nov 24, 2022. It is now read-only.

Cannot access email threads #8

Open

msp301 opened this issue Dec 12, 2019 · 1 comment

Labels

bug

Milestone

Owner

msp301 commented Dec 12, 2019

Describe the bug
The API returns lists of threads that contain messages. If we take the returned thread ID and give it to /api/messages/XXXXX it fails to do anything.

To Reproduce

GET /api/messages or GET /api/inbox
Take first any returned thread ID e.g. "0001"
GET /api/messages/0001

Expected behaviour
Get a successful response with maybe including the full thead info and some metadata for its messages

Actual behaviour
Error 500 - Failed to read message

The text was updated successfully, but these errors were encountered:

msp301 added the bug label

msp301 added this to the v1.0 milestone

Owner Author

msp301 commented Dec 12, 2019

This highlights a flaw in the API design.
I think we should be modelling the thread -> message relationship in the endpoints.
e.g.

GET /api/threads
GET /api/threads/<THREAD_ID>/messages
GET /api/threads/<THREAD_ID>/messages/<MESSAGE_ID>

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.