feat: CLUE support for URL query parameter + refactor public session link

thinkh commented 2 years ago

[x] Descriptive title for this pull request provided (will be used for release notes later)
[x] All acceptance criteria from the issue are met
[x] Branch is up-to-date with the branch to be merged with, i.e., develop
[x] Code is cleaned up and formatted
[ ] Code documentation written
[ ] Unit tests written
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Build is successful -> the build is failing because it depends on https://github.com/datavisyn/tdp_core/pull/762
[x] Summary of changes written
[ ] Wiki documentation written
[x] Assign at least one reviewer
[x] Assign at least one assignee
[x] Add type label (e.g., bug, enhancement) to this pull request
[x] Add next version label (e.g., next-version: minor) to this PR following semver

Refactor how the public session link is generated
Before it was hard-coded to use the hash property (#clue_graph=...)
With https://github.com/datavisyn/tdp_core/pull/762 the URL can use query parameter (?clue_graph=...)
Now the links are generated using the CLUE graph manager which decides whether hash or query should be used

grafik

In order to use the query parameter the following option in ATDPApplication must be set (default is hash):

const _ = new Ordino({
  cluePropertyHandler: 'query',
});

thinkh commented 2 years ago

@oltionchampari Thanks for testing and pointing it out. The empty hash (i.e., # only) is fixed with datavisyn/tdp_core@f10ab4d (#762).

thinkh commented 2 years ago

@oltionchampari please approve this PR so that we can merge the PR. Thanks.

Caleydo / ordino