Skip to content

Conversation

@juliusknorr
Copy link
Member

Fix #1916
Fix #1922

@stefan-niedermann
Copy link
Member

stefan-niedermann commented May 19, 2020

We have a share card url button in the android app. Are you aware of any change of the URL format across the different server versions?

@codecov-commenter
Copy link

codecov-commenter commented May 19, 2020

Codecov Report

Merging #1925 into master will decrease coverage by 0.00%.
The diff coverage is 62.50%.

@@            Coverage Diff             @@
##           master    #1925      +/-   ##
==========================================
- Coverage   56.92%   56.92%   -0.01%     
==========================================
  Files          77       77              
  Lines        3840     3842       +2     
==========================================
+ Hits         2186     2187       +1     
- Misses       1654     1655       +1     

@aszlig
Copy link

aszlig commented May 22, 2020

Doesn't this miss OCA\Deck\Activity\DeckProvider::deckUrl or are those bang-prefixed URL fragment identifiers still recognised?

@juliusknorr
Copy link
Member Author

We have a share card url button in the android app. Are you aware of any change of the URL format across the different server versions?

Yeah, unfortunately there is not much we can do there since vue-router has removed the support for the hashbang prefixing. Maybe we should provide a server side unique link to cards that can then adapt to whatever the frontend is doing (similar to the files apps /f/{fileid}).

Doesn't this miss OCA\Deck\Activity\DeckProvider::deckUrl or are those bang-prefixed URL fragment identifiers still recognised?

Ah nope, good catch.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliusknorr juliusknorr merged commit c6aa936 into master Jun 2, 2020
@juliusknorr juliusknorr deleted the bugfix/1922-1916 branch June 2, 2020 15:27
@stefan-niedermann
Copy link
Member

stefan-niedermann commented Jun 2, 2020

Yeah, unfortunately there is not much we can do there since vue-router has removed the support for the hashbang prefixing.

Okay, so there actually is a change. Can you please tell me how the link looks across the different server app versions, for example:

0.0.1 - 0.8.1: example.com/apps/deck/#!/board/5/card3
0.8.1 - 1.0.0: example.com/apps/deck/app/#!/board/5/card3
     >= 1.0.1: example.com/apps/deck/board/5/card3

Then i am able take care that copying URLs will work with each server version.

Maybe we should provide a server side unique link to cards that can then adapt to whatever the frontend is doing (similar to the files apps /f/{fileid}).

This makes sense to prevent broken bookmark links. But you should ensure that this won't result in additional request with a redirect for the web UI users.

@juliusknorr
Copy link
Member Author

See #1977 for the details and a fallback for the old routes.

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.

Notifications about deck comments cannot go to the right card because there is a extra symbol into the link Linking causes invalid URL

6 participants