-
Notifications
You must be signed in to change notification settings - Fork 1
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 tutorial screenshots #1
Conversation
Ahmad-Wahid
commented
May 18, 2023
- Added new asset view with and without data .
- New screenshot of battery discharge.
- Updated day-ahead-prices screenshot.
…arging screenshot Signed-off-by: Ahmad <ahmedwahid16101@gmail.com>
Signed-off-by: Ahmad <ahmedwahid16101@gmail.com>
Quick comment: If we rename screenshots, older versions of the documentation will not show properly anymore. In a perfect world, we'd find a way to manage the screenshots in a different way, but there is no perfect design we can think of. Why do we need screenshots without data anyway? |
I added it because, if you look at the first asset screenshot in the tutorial, it just shows the battery sensor, location, and editing options of the asset. |
…olar addition Signed-off-by: Ahmad <ahmedwahid16101@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Following up on my previous comment (which you did not reply to), I'm making some suggestions myself on how to keep older docs version working,
Signed-off-by: Ahmad <ahmedwahid16101@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good.
Note that in FlexMeasures/flexmeasures#687 I request one extra screenshot, so you might as well include it in this PR.
Signed-off-by: Ahmad <ahmedwahid16101@gmail.com>
Signed-off-by: Ahmad Wahid <59763365+Ahmad-Wahid@users.noreply.github.com>