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

Connect Story Data to Terria #100

Open
kretep opened this issue Aug 24, 2021 · 1 comment
Open

Connect Story Data to Terria #100

kretep opened this issue Aug 24, 2021 · 1 comment

Comments

@kretep
Copy link
Member

kretep commented Aug 24, 2021

As an Editor
I want to be able to add data to my story
so that I can show it on the show them in story pages.

Proposed Work

Every story in the database has a list of CatalogItems that should be loaded into terria when the story is loaded.
The data for every item should either be accessible through a url, or be uploaded to S3 through a dialog.

Todo:

  • The user should be able to add new data by uploading it from his computer

    • The existing data dialog needs to be designed
    • Adding data should upload it to S3
    • We probably want to keep the ability to use data from the local computer so editors can preview the data before uploading it.
      • Perhaps use this and upload to s3 when they press save (give a pop up that it will be uploaded)?
  • When loading a story

    • Add each item in the stories catalog to terria as catalog items
      • Check the addUserFiles function in terriajs/lib/Models/addUserFiles.js for logic on how to do this. We probably want to make a copy of this function to do this.
      • For each item we need to get the signed url from S3 to use in the catalog item so it can be loaded correctly
        • Before I made an S3Proxy to do this on the fly when encountering S3 urls, but I think when building the catalog is a better place.
@kretep kretep assigned kretep and bpmweel and unassigned kretep Aug 24, 2021
@bpmweel bpmweel changed the title Add S3 catalog group Connect Story Data to Terria Sep 3, 2021
@bpmweel
Copy link

bpmweel commented Sep 3, 2021

I have been working on parts of this for a while, instead of producing actual code that should be merged I have compiled my thoughts in the issue.

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

No branches or pull requests

3 participants