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

Docs/maxar demo review #133

Merged
merged 3 commits into from
Aug 25, 2023
Merged

Docs/maxar demo review #133

merged 3 commits into from
Aug 25, 2023

Conversation

zacharyDez
Copy link
Contributor

I've added a few things to the PR:

  • Quick review of some content (primarily typos)
  • Add an important note at the beginning of the notebook on authentication, getting started with eoAPI and endpoints being for demonstration purposes
  • Add a what's next section at the end of the notebook

Add important information note in introduction about demo endpoints, authentication, and getting started with eoAPI. Fix some typos in sections I had not reviewed.
@zacharyDez zacharyDez added the documentation Improvements or additions to documentation label Aug 25, 2023
@zacharyDez
Copy link
Contributor Author

@vincentsarago will merge into the main, and we can review with the #132 afterward. It'll be easier to review both at the same time.

@zacharyDez zacharyDez merged commit c9f1d1f into main Aug 25, 2023
2 checks passed
@zacharyDez zacharyDez deleted the docs/maxar-demo-review branch August 25, 2023 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants