Skip to content

Comments

Content: fire#260

Merged
j08lue merged 1 commit intoNASA-IMPACT:content/firefrom
mccabete:eis_fire_content
Jun 21, 2023
Merged

Content: fire#260
j08lue merged 1 commit intoNASA-IMPACT:content/firefrom
mccabete:eis_fire_content

Conversation

@mccabete
Copy link
Contributor

Adding content

FEDS fire Perimeter description, with links to datasets based on FEDS fire perimeters. This is a first draft, that hasn't yet had feedback from wider EIS fire team. I will want to update the initial image, and will have more/different content. This should start a good group conversation though.

Any content

  • Cover image with proper attribution (when relevant) is present.
  • Cover images do not contain any text (they're used as a background and text may not be readable).
  • All images are not too big/heavy. As a general rule of thumb, they should not weigh more than 500KB, or be larger than 2000px. See the media guide for more info.
  • Content was added using the available MDX Blocks.

Datasets

@netlify
Copy link

netlify bot commented May 16, 2023

Deploy Preview for visex ready!

Name Link
🔨 Latest commit e3613f0
🔍 Latest deploy log https://app.netlify.com/sites/visex/deploys/6463e5fb855b1c0008c1bace
😎 Deploy Preview https://deploy-preview-260--visex.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

Copy link
Collaborator

@danielfdsilva danielfdsilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mccabete Looks good overall. Perhaps the map layer description could be improved. I imagine it could be the same as the dataset?

@mccabete
Copy link
Contributor Author

@danielfdsilva -- that's a great point. Could you point me to where I would edit that? For some reason I assumed that I could n't edit that section.

@danielfdsilva
Copy link
Collaborator

@mccabete It's on that same file. Lines 21 and 23 I think.

@danielfdsilva
Copy link
Collaborator

@mccabete Anything we can help with to get this merged?

@mccabete
Copy link
Contributor Author

mccabete commented May 31, 2023 via email

@j08lue
Copy link
Contributor

j08lue commented Jun 21, 2023

This is just a PR onto another content branch. I'll merge this to reduce the number of PRs, we can track the evolution of this content on the other PR (#234).

@j08lue j08lue merged commit a042757 into NASA-IMPACT:content/fire Jun 21, 2023
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

Successfully merging this pull request may close these issues.

3 participants