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

replace consecutive identical logs with single log having count #123

Merged
merged 5 commits into from
Apr 20, 2016
Merged

replace consecutive identical logs with single log having count #123

merged 5 commits into from
Apr 20, 2016

Conversation

ritz078
Copy link
Contributor

@ritz078 ritz078 commented Apr 19, 2016 •
edited
Loading

Just another improvement in the action logger.

screen shot 2016-04-19 at 10 53 53 pm

@ritz078 ritz078 changed the title replace consecutive identical logs with single log and count replace consecutive identical logs with single log having count Apr 19, 2016
arunoda
arunoda reviewed Apr 20, 2016
View reviewed changes
src/client/ui/action_logger.js
@@ -40,6 +40,7 @@ const btnStyle = {
class ActionLogger extends Component {

getActionData() {
// const actions = formatActionData(this.props.actions);
Copy link
Member

Choose a reason for hiding this comment

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

Let's remove this.

@arunoda
Copy link
Member

arunoda commented Apr 20, 2016

This is super. I'll take this now.
No need to do the changes, I've mentioned.
I'll do it now.

@arunoda arunoda merged commit c563ea2 into storybookjs:master Apr 20, 2016
@arunoda
Copy link
Member

arunoda commented Apr 20, 2016

Released with v1.15.0.

@ritz078 ritz078 deleted the feat/action-logger branch April 25, 2016 06:49
ndelangen pushed a commit that referenced this pull request Apr 5, 2017
Merge pull request #123 from fson/patch-1
335af94 
Update port number in README
@shilman shilman added the misc label May 27, 2017
@dependencies dependencies bot mentioned this pull request Sep 30, 2017
Merged
@chakAs3 chakAs3 mentioned this pull request May 22, 2023
Merged
5 tasks
@nx-cloud Nx Cloud
Copy link

nx-cloud bot commented Mar 20, 2024 •
edited
Loading

View your CI Pipeline Execution ↗ for commit c563ea2.

Command Status Duration Result
nx run-many -t build --parallel=3 ✅ Succeeded 1m 36s View ↗

☁️ Nx Cloud last updated this comment at 2024-12-22 21:05:51 UTC

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ritz078 @arunoda @shilman