Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Cherry-pick(s): allow min/max value , show y-axis for sparklines in time series table #83

Merged
merged 1 commit into from
Aug 14, 2018

Conversation

kristw
Copy link

@kristw kristw commented Aug 14, 2018

Cherry-pick the following commit(s):

SHA Time Log Author
536478e Mon Aug 13 16:19:19 2018 -0700 [feature] Allow min/max value for the sparkline in time series table (apache#5603) (apache/master) [Krist Wongsuphasawat]

@graceguo-supercat @michellethomas @john-bodley @williaster

…pache#5603)

* Allow min/max value for the sparkline in time series table

* show bound lines

* User can choose to show y-axis bounds

* show label for the bounds

* compute necessary padding for the bound label

* extract sparkline code to another component

* can show y-axis in sparkline without setting bounds

* reorder option rows

(cherry picked from commit 536478e)
@kristw kristw changed the title [WIP] Cherry-pick(s): <description> Cherry-pick(s): allow min/max value , show y-axis for sparklines in time series table Aug 14, 2018
@john-bodley
Copy link
Collaborator

@kristw thanks for putting together the table but in the future it's probably not necessary as GitHub lists all the commit which have been cherry-picked.

@kristw kristw merged commit de73024 into airbnb:airbnb-master Aug 14, 2018
@williaster
Copy link

FWIW I dig the table! :)

I take it you're writing a script for this?! 💯 If so, great idea / we should all use that or automate it, this takes way too much time!

@kristw
Copy link
Author

kristw commented Aug 14, 2018

@john-bodley @williaster Yes, I wrote a shell script for this (create PR from command line and pipe git log into commit message). :) Revising it a bit more and will share with the team.

@kristw kristw deleted the kristw/sparkline branch August 15, 2018 00:28
michellethomas pushed a commit that referenced this pull request Dec 1, 2021
michellethomas pushed a commit that referenced this pull request Dec 1, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants