-
Notifications
You must be signed in to change notification settings - Fork 139
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
Cicero UI -> Web Components #354
Conversation
docs/ref-web-components-overview.md
Outdated
|
||
You can preview these components in [the project's storybook](https://ap-web-components.netlify.app/). | ||
|
||
![Template-Studio-V2](/docs/assets/reference/tsv2.png) |
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.
Although I'm not a maintainer I apologize for commenting here but, do we need this image which is of template-studio-v2
(which is archived now)?
Link of this PR preview build : https://deploy-preview-354--accordproject-old.netlify.app/docs/next/ref-web-components-overview.html
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.
Yeah..I need to update the image!!
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.
I think we don't really need an image here at the moment, so this could be removed.
docs/started-resources.md
Outdated
### User Interface Components | ||
|
||
- [Markdown Editor](https://github.com/accordproject/web-components/tree/master/packages/ui-markdown-editor): a general purpose react component for markdown rendering and editing | ||
- [Cicero UI](https://docs.accordproject.org/docs/ref-cicero-ui.html): a library of react components for visualizing, creating and editing Accord Project templates |
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.
"Cicero UI" does not exist anymore, so I think we want to remove this reference and expand on what currently exists in web-components
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.
You'll want to ensure you provide a DCO sign-off for your commits using the --signoff
option of git commit. This will require a rebase, in which you can reword your commit messages to follow the AP format
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.
Can you remove docs/assets/reference/sb.png
?
docs/started-resources.md
Outdated
### User Interface Components | ||
|
||
- [Markdown Editor](https://github.com/accordproject/web-components/tree/master/packages/ui-markdown-editor): a general purpose react component for markdown rendering and editing | ||
- [Contract Editor](https://github.com/accordproject/web-components/tree/master/packages/ui-contract-editor):provides a rich-text content editor for contract text with embedded clauses. |
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.
Can you make this:
[Contract Editor](https://github.com/accordproject/web-components/tree/master/packages/ui-contract-editor): a rich-text content editor for contract text with embedded clauses.
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.
Do you mean to change the content or develop the page?
Signed-off-by: Aniruddha Shriwant <[email protected]> Signed-off-by: My Name <[email protected]>
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.
@rutvi18 Here first of all make a git pull origin issue#301
and then use the command.It will work fine.
Signed-off-by: Aniruddha Shriwant <[email protected]> Signed-off-by: My Name <[email protected]>
Signed-off-by: My Name <[email protected]>
Signed-off-by: My Name <[email protected]>
Signed-off-by : Rutvi Sharma <[email protected]> Signed-off-by: My Name <[email protected]>
Signed-off-by: Rutvi Sharma <[email protected]> Signed-off-by: My Name <[email protected]>
Signed-off-by: Rutvi Sharma <[email protected]> Signed-off-by: My Name <[email protected]>
Signed-off-by: Rutvi Sharma Signed-off-by: My Name <[email protected]>
Signed-off-by: Rutvi Sharma [email protected] Signed-off-by: My Name <[email protected]>
Signed-off-by: My Name <[email protected]>
Signed-off-by: My Name <[email protected]>
Signed-off-by: My Name <[email protected]>
Hey @rutvi18, |
Since this change is really small, could you rebase the 26 commits down to 1 commit please? You can then also ensure you provide a DCO sign-off for your commits using the --signoff option of git commit, as well as rewording your commit messages to follow the AP format. |
author My Name <[email protected]> 1617033576 +0530 committer My Name <[email protected]> 1617121228 +0530 parent e4a788b author My Name <[email protected]> 1617033576 +0530 committer My Name <[email protected]> 1617121227 +0530 parent e4a788b author My Name <[email protected]> 1617033576 +0530 committer My Name <[email protected]> 1617121224 +0530 changes_done Signed-off-by: My Name <[email protected]> Updated the started-resources.md Signed-off-by : Rutvi Sharma <[email protected]> Signed-off-by: My Name <[email protected]> Updated the page Signed-off-by: Rutvi Sharma <[email protected]> Signed-off-by: My Name <[email protected]> Updated picture Signed-off-by: Rutvi Sharma Signed-off-by: My Name <[email protected]> image removed Signed-off-by: My Name <[email protected]> deleted the image Update started-resources.md Signed-off-by: My Name <[email protected]>
@rutvi18 any update on this? |
Closes #301
Changes
Screenshots or Video
Author Checklist
--signoff
option of git commit.master
fromfork:Issue#301