Skip to content
This repository has been archived by the owner on May 19, 2020. It is now read-only.

feat(*): add storybook for developer focused demo - I315 #353

Merged
merged 2 commits into from
Mar 20, 2020

Conversation

sanyamdogra
Copy link
Contributor

Issue #315

Changes

  • Adds @storybook/react as a dependency.
  • Access Storybook at port 9001 by running npm run storybook.
  • Component stories can be added in src/stories folder with extension .stories.js

Flags

  • N.A

Related Issues

Screenshots

image
image

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
@jolanglinais
Copy link
Member

This is done with @elit-altum as far as I understand.

Copy link
Member

@DianaLease DianaLease left a comment

Choose a reason for hiding this comment

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

LGTM

@jeromesimeon
Copy link
Member

This is done with @elit-altum as far as I understand.

Can you clarify this? Who wrote the code, and who signed the DCO?

@jolanglinais
Copy link
Member

@jeromesimeon see the work in #335

Copy link
Member

@mttrbrts mttrbrts left a comment

Choose a reason for hiding this comment

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

@irmerk Let's get a Netlify build of this running once this is merged

@jolanglinais jolanglinais changed the title feat(*): Adds Storybook for Developer Focused Demo - I315 feat(*): add storybook for developer focused demo - I315 Mar 20, 2020
@jolanglinais jolanglinais merged commit 41578ee into accordproject:development Mar 20, 2020
jolanglinais pushed a commit that referenced this pull request Apr 10, 2020
* feat(*): Adds Storybook deps and inital configs  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>

* feat(*): Adds Dummy stories for ContractEditor and ErrorLogger  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
jolanglinais pushed a commit to jolanglinais/cicero-ui that referenced this pull request Apr 10, 2020
…ct#353)

* feat(*): Adds Storybook deps and inital configs  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>

* feat(*): Adds Dummy stories for ContractEditor and ErrorLogger  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
Signed-off-by: irmerk <jolenelanglinais@gmail.com>
jolanglinais pushed a commit that referenced this pull request Apr 10, 2020
* feat(*): Adds Storybook deps and inital configs  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>

* feat(*): Adds Dummy stories for ContractEditor and ErrorLogger  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
Signed-off-by: irmerk <jolenelanglinais@gmail.com>
jolanglinais pushed a commit to jolanglinais/cicero-ui that referenced this pull request Apr 10, 2020
…ct#353)

* feat(*): Adds Storybook deps and inital configs  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>

* feat(*): Adds Dummy stories for ContractEditor and ErrorLogger  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
@jolanglinais
Copy link
Member

@sanyamdogra can you check to ensure these changes are still in master?

jolanglinais pushed a commit that referenced this pull request Apr 10, 2020
* feat(*): Adds Storybook deps and inital configs  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>

* feat(*): Adds Dummy stories for ContractEditor and ErrorLogger  - I315

Signed-off-by: Sanyam Dogra <sdlord07@gmail.com>
@sanyamdogra
Copy link
Contributor Author

Hi @irmerk, Yes they have been added to the master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants