mirror of
https://github.com/dani-garcia/vaultwarden.git
synced 2026-04-25 17:25:57 +03:00
[GH-ISSUE #100] PUT /api/ciphers/share not implemented #57
Labels
No labels
SSO
Third party
better for forum
bug
bug
documentation
duplicate
enhancement
future Vault
future Vault
future Vault
good first issue
help wanted
low priority
notes
pull-request
question
troubleshooting
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/vaultwarden#57
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
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?
Originally created by @mprasil on GitHub (Jul 21, 2018).
Original GitHub issue: https://github.com/dani-garcia/vaultwarden/issues/100
New vault (#93) calls
PUTon/api/ciphers/shareto "Share selected" ciphers. This functionality wasn't present in the previous vault so implementation will be a bit more involved, but should be pretty similar to "move selected".(also woohoo! 100th issue 😄)
@tgturner commented on GitHub (Jul 26, 2018):
I would love to take a look at this one!
@mprasil commented on GitHub (Jul 26, 2018):
Go for it and don't hesitate to ask for help.
@tgturner commented on GitHub (Jul 26, 2018):
Thanks! I'll take a look at it this evening.
@krankur commented on GitHub (Aug 1, 2018):
I guess we need to add an organization in order to test the "Share" functionality, but I am not able to figure out how to add an organization? I tried uploading a random JSON file as the license file but I am getting a 401 error. Can someone please guide me on this?
@mprasil commented on GitHub (Aug 1, 2018):
Hi, there's no licence required - just create free organization (the first option) and it should have all features after that.
@dani-garcia commented on GitHub (Aug 1, 2018):
I think that happens in the self hosted version (when you build with npm run dist:selfhost)
It should work if you build the normal version (with npm run dist)
@mprasil commented on GitHub (Aug 1, 2018):
@dani-garcia I submitted #109 to merge recent changes from master. We should do this before @krankur touches the sharing API as there were some major changes and will be pain to merge later.
@mprasil commented on GitHub (Aug 1, 2018):
Thanks for merging it. @krankur make sure you're working with latest beta code.
@krankur commented on GitHub (Aug 1, 2018):
@mprasil Will do. Thanks for the heads up.
@mprasil commented on GitHub (Aug 9, 2018):
I think this was resolved in #116