Make UserNote comment default to the empty string. #530

Merged
floatingghost merged 3 commits from provable_ascent/akkoma:provable_ascent-patch-1 into develop 2023-05-22 21:33:02 +00:00
Contributor

This makes the behavior consistent between when UserNote doesn't exist and when comment is null.

The current behavior may return null in APIs, which misleads some clients doing feature detection into thinking the server does not support comments.
For example, see https://codeberg.org/husky/husky/issues/92

This makes the behavior consistent between when UserNote doesn't exist and when comment is null. The current behavior may return null in APIs, which misleads some clients doing feature detection into thinking the server does not support comments. For example, see https://codeberg.org/husky/husky/issues/92
Make UserNote comment default to the empty string.
Some checks are pending
ci/woodpecker/pr/woodpecker Pipeline is pending
d8bed0ff63
This make the behavior consistent between when UserNote doesn't exist and when comment is null.

The current behavior may return null in APIs, which misleads some clients doing feature detection into thinking the server does not support comments.
For example, see https://codeberg.org/husky/husky/issues/92

can you add a test to this to confirm that it works as you expect it to? i don't see why it shouldn't but hey

can you add a test to this to confirm that it works as you expect it to? i don't see why it shouldn't but hey
Add user_note_test.exs.
Some checks are pending
ci/woodpecker/pr/woodpecker Pipeline is pending
9c4203632d
Author
Contributor

@floatingghost Test added! PTAL

@floatingghost Test added! PTAL

this is good, thanks a lot!

this is good, thanks a lot!
floatingghost deleted branch provable_ascent-patch-1 2023-05-22 21:33:02 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
AkkomaGang/akkoma!530
No description provided.