Skip to content

Transition on latest logged action.#80

Merged
arunoda merged 3 commits intostorybookjs:masterfrom
ritz078:fix/log-transition
Apr 9, 2016
Merged

Transition on latest logged action.#80
arunoda merged 3 commits intostorybookjs:masterfrom
ritz078:fix/log-transition

Conversation

@ritz078
Copy link
Copy Markdown
Contributor

@ritz078 ritz078 commented Apr 7, 2016

More details in #51

Comment thread src/client/ui/admin.js Outdated
.join('\n\n');

return (<ActionLogger actionLog={log} onClear={clearLogs} />);
return (<ActionLogger data={data} onClear={clearLogs} />);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Here do not send the whole data, but just send an array of strings. We can do it like this:

export function getActionLogger(data) {
  const { actions = [] } = data;
  const logs = actions
    .map(action => stringify(action, null, 2));

  return (<ActionLogger logs={logs} onClear={clearLogs} />);
}

@arunoda arunoda merged commit 27d7f94 into storybookjs:master Apr 9, 2016
@arunoda
Copy link
Copy Markdown
Member

arunoda commented Apr 9, 2016

Thanks.

@ritz078 ritz078 deleted the fix/log-transition branch April 9, 2016 18:44
@shilman shilman added the misc label May 27, 2017
ndelangen pushed a commit that referenced this pull request Feb 23, 2024
@nx-cloud
Copy link
Copy Markdown

nx-cloud bot commented Jun 19, 2024

View your CI Pipeline Execution ↗ for commit 27d7f94

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

☁️ Nx Cloud last updated this comment at 2025-09-18 02:10:03 UTC

kasperpeulen pushed a commit that referenced this pull request Feb 6, 2025
Add utility for tag combination & negation
yannbf added a commit that referenced this pull request Apr 17, 2025
fix: add optional chaining to object property key
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants