To to list: May-June - Githubissues

ModelAtlasofTheEarth / website

website for Model Atlas of the Earth

MIT License

0 stars 2 forks source link

To to list: May-June #61

Open dansand opened 3 months ago

dansand commented 3 months ago

[x] In the snapshot tab, if no animation, revert to figure
[x] Add "how to cite", based on the creditText field
- [ ] See PR #78 - @cpalfonso
[x] What sections currently support markdown/latex formatting, and is it possible to make this apply to all text sections and captions?
[ ] Software section of YAML does not get imported properly, and my attempts to fix this have led to many conflicts.
[ ] Need to add Funder/Funding information to the "about" tab
[ ] Need MATE collection DOI on landing page (http://dx.doi.org/10.25914/yrzp-g882)

cpalfonso commented 3 months ago

I've been unable to build the website locally (interestingly, Netlify doesn't seem to be having any issues), and I've managed to figure out the cause: if no animation is uploaded, the animation field in the markdown file just points directly to the graphics directory: https://github.com/ModelAtlasofTheEarth/website/blob/46353735a6227f9d77822c13b1309c0fc724db3a/src/pages/models/mather-2022-groundwater/index.md?plain=1#L143-L145 Gatsby then throws an error because this isn't a file. Maybe we should just remove the animation field entirely if one is not provided?

cpalfonso commented 3 months ago

One more thing I've just spotted: whatever process is creating the creditText field in the markdown files isn't grabbing the whole DOI, only the second half:

https://github.com/ModelAtlasofTheEarth/website/blob/dbdf4cc4865aeb14a92c1c9b5032f2b86ffaad25/src/pages/models/RodriguezCorcho-2022-ArcCollision/index.md?plain=1#L12

https://github.com/ModelAtlasofTheEarth/website/blob/dbdf4cc4865aeb14a92c1c9b5032f2b86ffaad25/src/pages/models/RodriguezCorcho-2022-ArcCollision/index.md?plain=1#L14-L17

dansand commented 3 months ago

Hey @cpalfonso. Thanks for this.

I've been working on some updates, but I've hit some problems with the software section. Currently this does not get loaded properly into the model.js, and my attempts to update this have led to lots of issues, complaints about conflicts. It would be great if you could take a look. Currently the software section is formatted as an array. We could chnage this, but it gives some flexibility later.