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

Fix generated fronted urls #1925

Merged
merged 1 commit into from
Jun 2, 2020
Merged

Fix generated fronted urls #1925

merged 1 commit into from
Jun 2, 2020

Conversation

juliushaertl
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?

@juliushaertl
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>
@juliushaertl juliushaertl merged commit c6aa936 into master Jun 2, 2020
@juliushaertl juliushaertl 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.

@juliushaertl
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
5 participants