Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable21] Add sharing deck card with a conversation #5174

Closed
wants to merge 3 commits into from

Conversation

backportbot-nextcloud[bot]
Copy link

@backportbot-nextcloud backportbot-nextcloud bot commented Feb 15, 2021

⚠️ This backport had conflicts and is incomplete ⚠️

backport of #5110

@nickvergessen
Copy link
Member

11.1 only

@PVince81 PVince81 self-assigned this Feb 15, 2021
@PVince81
Copy link
Member

I'll solve the conflicts

PVince81 and others added 3 commits February 15, 2021 10:58
Added deck action for sharing/linking that uses the RoomSelector.
Improved RoomSelector

Co-authored-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Deck plugin now creates its own Vue VM to instantiate the
RoomSelector.
RoomSelector now allows to specify properties for dialog title and
whether the result list should be filtered.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Expose the Talk Vue instance on OCA.Talk.instance.
In RoomSelector, grab the current room from that instance to filter it
out, when applicableRemove current room from RoomSelector

Expose the Talk Vue instance on OCA.Talk.instance.
In RoomSelector, grab the current room from that instance to filter it
out, when applicable.

Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Copy link
Member

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

Seems we're missing the endpoint, getting a 404 on:

xhr.js:177 POST https://vvortex.local/ocs/v2.php/apps/spreed/api/v1/chat/sk7eh2fs/share 404 (Not Found)

@PVince81
Copy link
Member

conflicts solved, retested => 💥 see above

@PVince81 PVince81 closed this Feb 15, 2021
@nickvergessen nickvergessen deleted the backport/5110/stable21 branch July 27, 2023 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants