mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-07-16 08:54:01 -05:00
[PR #1881] [MERGED] Fix sends notification format and add endpoints to get all sends or one send by id #6660
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
📋 Pull Request Information
Original PR: https://github.com/dani-garcia/vaultwarden/pull/1881
Author: @fabianthdev
Created: 8/3/2021
Status: ✅ Merged
Merged: 8/22/2021
Merged by: @dani-garcia
Base:
main← Head:fix/sends_notifications📝 Commits (3)
dd98fe8Send create, update and delete notifications forSends in the correct format.42ba817Fix errors that occurred in the nightly build1f0f64dSort the imports in notifications.rs alphabetically📊 Changes
3 files changed (+58 additions, -10 deletions)
View changed files
📝
src/api/core/sends.rs(+35 -7)📝
src/api/notifications.rs(+18 -1)📝
src/db/models/send.rs(+5 -2)📄 Description
This small pull request fixes the notification format for
SyncSendCreate,SyncSendUpdateandSyncSendDeletenotification types.With the correct notification format, the sends in the web vault and other bitwarden clients are automatically updated when the user creates, updates or deletes a send in another session. To allow the clients to update specific sends, this PR also includes new endpoints for getting all sends or one specific send by uuid.
I've verified that this works with the web vault and the bitwarden mobile client.
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.