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

Update dashboard contributing documentation #179

Merged
merged 8 commits into from
Nov 11, 2024

Conversation

aboydnw
Copy link
Contributor

@aboydnw aboydnw commented Nov 7, 2024

The docs for contributing to the Dashboard are out of date. I also changed the name from "contributing" to "data and content configuration" to avoid possible confusion about contributing to core VEDA code. This section is more for managing content of somebody's specific instance.

Copy link

github-actions bot commented Nov 7, 2024

PR Preview Action v1.4.8
🚀 Deployed preview to https://NASA-IMPACT.github.io/veda-docs/pr-preview/pr-179/
on branch gh-pages at 2024-11-11 10:49 UTC

@aboydnw aboydnw self-assigned this Nov 8, 2024
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Contributor

@j08lue j08lue left a comment

Choose a reason for hiding this comment

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

Good call to rename the "Contributing" section to clarify that it is about content, not software.

I suggest we call it "Adding Content" for brevity, including both data and prose.

image

Good to see an update to the Dashboard contribution docs. There is a lot of duplication between Dataset and Story configuration docs now, but I guess that is okay.

@aboydnw aboydnw merged commit ddafbc6 into main Nov 11, 2024
1 check passed
@aboydnw
Copy link
Contributor Author

aboydnw commented Nov 11, 2024

Good call @j08lue ! Much simpler title

@j08lue j08lue deleted the update-dashboard-contributing branch November 11, 2024 19:51
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.

2 participants