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

Story v2.0 - Placeholder routes for "new stories" via /stories/:id #4692

Closed
Tracked by #111
AnaBelgun opened this issue Sep 1, 2020 · 13 comments
Closed
Tracked by #111

Story v2.0 - Placeholder routes for "new stories" via /stories/:id #4692

AnaBelgun opened this issue Sep 1, 2020 · 13 comments
Assignees
Labels
Version 8 formerly MobX

Comments

@AnaBelgun
Copy link
Member

AnaBelgun commented Sep 1, 2020

Background:
DEA team would like to start using vanity url shortner for broadcasting work on some specific datasets.
The first example will be the use of the short url for the Coastlines data, to be published and disseminated to Government and other users in Sep.

There are 2 stages:

  1. Once DEA team provides a share link with the story they want, we'll copy the contents of that share link into a cleaner s3 directory - remembering we prefix the first 2 characters into directories - https://github.com/TerriaJS/terriajs-server/blob/f2aa63e95debfdb4885ddacdfa7dffef2dde8d29/lib/controllers/share.js#L103. That way, we can make https://maps.dea.ga.gov.au/#share=s-DEACoastlines resolve to that share link

  2. The placeholder feature described in this actual ticket title - once the new routing in 1757 mobx new routing #4679 is merged, we will add an additional route under /stories/:id, which will form the basis of the new stories feature extensions. But the "placeholder" will simply resolve the :id into a s3 share link under the same key (we do S3 here because realistically that's all we are using at the moment)

1) enable the http://maps.dea.ga.gov.au/**#share=s-DEACoastlines** when DEA team advises the data publication and its url

2) work and release a more elegant shortening of urls which will transform the above url closer to http://maps.dea.ga.gov.au/**DEACoastlines**

@hardreddata
Copy link
Contributor

In the first instance I wonder if you could do this with initialisation files and so http://maps.dea.ga.gov.au/#DEACoastlines would be the link, assuming you put an equivalent DEACoastlines.json in wwwroot/init

Not really my business, but potentially a good compromise for a September time constraint. If they can maintain their own catalogue.

This is how the branch landing pages referred to in https://medium.com/terria/engineering-excellence-with-terria-douglas-partners-9da433753f23 are done

@soyarsauce
Copy link
Contributor

ok so here was my thinking for /stories/DEACoastlines

  • if they were going to publicise some url broadly, we should take this opportunity to do something nice like above
  • having that lets us do /stories/DEACoastlines to the share link today, then down the future that same link can be stories v2
  • and doing the /stories/DEACoastline is like, a 5 min job over the 1min-s3 job

Will update ticket description to clarify we aren't aiming for a url shortener in Terria

@RussellGrew that's actually a good idea! If not for the fact that this will be utilising the story feature and not just a predefined catalog

@soyarsauce soyarsauce changed the title Develop option to create a vanity url shortner Mobx: Placeholder routes for "new stories" via /stories/:id Sep 2, 2020
@soyarsauce soyarsauce added the Version 8 formerly MobX label Sep 2, 2020
@hardreddata
Copy link
Contributor

As you may expect we are very happy to take receipt of this new feature. All good. Thanks for the response.

@soyarsauce
Copy link
Contributor

Fantastic 🙂

@RussellGrew FYI points 1 & 2 on the issue description are updated now

@meh9
Copy link
Contributor

meh9 commented Sep 2, 2020

In the first instance I wonder if you could do this with initialisation files and so http://maps.dea.ga.gov.au/#DEACoastlines would be the link, assuming you put an equivalent DEACoastlines.json in wwwroot/init

Hi @RussellGrew - yes you absolutely can, we used to do exactly that for AREMI in the past. If you use initFragmentPaths in config.json then you can have a non-wwwroot/init space where you put your additional init files. We used to keep ours in a static S3 web bucket because it was very easy to add new ones. However, this was before Stories, so not sure if it would work for them or not.

@robbibt
Copy link

robbibt commented Sep 2, 2020

http://maps.dea.ga.gov.au/#DEACoastlines would be pretty fantastic, we'd love to take up that option if it was possible!

@AnaBelgun
Copy link
Member Author

Blocked by #4961

@AnaBelgun AnaBelgun changed the title Mobx: Placeholder routes for "new stories" via /stories/:id Stpry v2.0 - Placeholder routes for "new stories" via /stories/:id Jul 1, 2021
@AnaBelgun AnaBelgun changed the title Stpry v2.0 - Placeholder routes for "new stories" via /stories/:id Story v2.0 - Placeholder routes for "new stories" via /stories/:id Sep 29, 2021
@wibily
Copy link
Contributor

wibily commented Feb 2, 2022

Ana: need a conclusion for this ticket by march

@wibily
Copy link
Contributor

wibily commented Feb 2, 2022

decision in team meeting to create static urls for stories so that customers (starting with DEA) can url shorten them and share them. Decision to not link this to SEO/route reserving.
Requires discussion on who is reponsible for generation of fancy share links. Possibilty of delegating this to another service.

@wibily
Copy link
Contributor

wibily commented Feb 2, 2022

Ana: these share links will be created manually and curated and maintained by the Terria team.

@steve9164
Copy link
Member

See draft here: #6151

Still deciding on route name (/story/{storyname} or /stories/{storyname}) and whether to immediately drop /story/{storyname} from the URL once the story loads, or whether to keep it (for page refreshes etc.)

@AnaBelgun
Copy link
Member Author

Connected to TerriaJS/TerriaMap#578 too

@AnaBelgun
Copy link
Member Author

both #6151 and TerriaJS/TerriaMap#578 have been merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Version 8 formerly MobX
Projects
None yet
Development

No branches or pull requests

7 participants