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.

Sign up for GitHub

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

Jump to bottom

Update documentation #460

Merged
merged 74 commits into from
Nov 20, 2023
Merged

Update documentation #460

merged 74 commits into from
Nov 20, 2023

Conversation

chrwm
Copy link
Member

@chrwm chrwm commented Aug 7, 2023 •
edited by FlorianK13
Loading

Summary of the discussion

tbd

Automation

Closes #434

PR-Assignee

Reviewer

  • 🐙 Follow the Reviewer Guidelines
  • 🐙 Provided feedback and show sufficient appreciation for the work done

@chrwm chrwm added 📖 documentation Improvements or additions to documentation 🚀 feature New feature or request labels Aug 7, 2023
@chrwm chrwm marked this pull request as ready for review September 26, 2023 09:14
@chrwm chrwm requested a review from FlorianK13 September 26, 2023 09:14
FlorianK13 and others added 2 commits September 27, 2023 09:56
@FlorianK13
Merge branch 'develop' into feature-434-documentation
31881bc
@FlorianK13
Uncomment create_additional_data_requests
559066d 
As various docstrings refer to this function, it seems at it is
intended to call this function directly by the user. Hence I uncommented
it again.
@FlorianK13
Copy link
Member

@chrwm @Ludee is the data_release_notes still maintained and up to date? Hence, should it be included in the documentation page?

@FlorianK13
Copy link
Member

Finish #467 and decide how the download functions with API and bulk are presented at the "Getting Started" page

@FlorianK13 FlorianK13 linked an issue Nov 2, 2023 that may be closed by this pull request
Update the documentation #434
Closed
16 tasks
@FlorianK13
Copy link
Member

Documentation branch is hosted at https://open-mastr.readthedocs.io/en/feature-434-documentation/

@FlorianK13 FlorianK13 marked this pull request as draft November 3, 2023 15:35
@FlorianK13 FlorianK13 marked this pull request as ready for review November 20, 2023 12:18
FlorianK13
FlorianK13 approved these changes Nov 20, 2023
View reviewed changes
Copy link
Member

@FlorianK13 FlorianK13 left a comment

Choose a reason for hiding this comment

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

After reviewing the created documentation page on readthedocs I can say it looks fine.

@FlorianK13 FlorianK13 merged commit 6402b39 into develop Nov 20, 2023
6 checks passed
@FlorianK13 FlorianK13 deleted the feature-434-documentation branch November 20, 2023 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@FlorianK13 FlorianK13 FlorianK13 approved these changes

Assignees

@chrwm chrwm

@FlorianK13 FlorianK13

Labels
📖 documentation Improvements or additions to documentation 🚀 feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update the documentation
2 participants
@chrwm @FlorianK13