Skip to content

Conversation

@nickvergessen
Copy link
Member

Fix #3523

Before After
Bildschirmfoto von 2020-05-13 16-11-16 Bildschirmfoto von 2020-05-13 16-11-13

In case we decide to backport this to stable18, the route needs to be changed.

@MartinKoeditz
Copy link

Just a hint:

I added your code change (the one liner) and now I receive the link. But it points to the nextcloud main page rather than to the room. Maybe it depends on my current version. Otherwise please have a look at that.

@nickvergessen
Copy link
Member Author

Yeah that is the line bellow the image.

For Nextcloud 18 the line would need to be something like:
'link' => $this->url->linkToRouteAbsolute('spreed.pagecontroller.showCall', ['token' => $room->getToken()]),

@nickvergessen nickvergessen force-pushed the bugfix/3523/make-conversations-linked-in-activities branch from 35008a7 to 78c9a72 Compare May 14, 2020 06:11
@nickvergessen nickvergessen force-pushed the bugfix/3523/make-conversations-linked-in-activities branch from 78c9a72 to 5eb3dba Compare May 14, 2020 07:39
@nickvergessen
Copy link
Member Author

/backport to stable19

@nickvergessen nickvergessen merged commit da6bdf1 into master May 14, 2020
@nickvergessen nickvergessen deleted the bugfix/3523/make-conversations-linked-in-activities branch May 14, 2020 14:41
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.

Missing invitation e-mail for registered users

4 participants