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.

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

chore(deps): cleaned up irrelevant dependencies in ui-markdown-editor #290

Closed
wants to merge 3 commits into from

Conversation

Cronus1007
Copy link
Contributor

@Cronus1007 Cronus1007 commented Mar 11, 2021

Closes #52

Changes

  • Removed rimraf package
  • Removed npm-run-all package

Flags

Screenshots or Video

Related Issues

Author Checklist

  • Ensure you provide a DCO sign-off for your commits using the --signoff option of git commit.
  • Vital features and changes captured in unit and/or integration tests
  • Commits messages follow AP format
  • Extend the documentation, if necessary
  • Merging to master from fork:branchname
  • Manual accessibility test performed
    • Keyboard-only access, including forms
    • Contrast at least WCAG Level A
    • Appropriate labels, alt text, and instructions

@mttrbrts mttrbrts changed the title cleaned up irrelevant dependencies in ui-markdown-edito chore(deps): cleaned up irrelevant dependencies in ui-markdown-editor Mar 12, 2021
@Cronus1007
Copy link
Contributor Author

@irmerk @mttrbrts Any updates?

@Cronus1007 Cronus1007 closed this Mar 15, 2021
@Cronus1007 Cronus1007 deleted the issue- branch March 15, 2021 15:55
@Cronus1007 Cronus1007 restored the issue- branch March 15, 2021 15:56
@Cronus1007 Cronus1007 reopened this Mar 15, 2021
Copy link
Member

@jolanglinais jolanglinais left a comment

Choose a reason for hiding this comment

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

Needs investigation by @accordproject/maintainers-ui

@Cronus1007
Copy link
Contributor Author

@irmerk I have rebase the commits. Any updates regarding this PR?

@jolanglinais
Copy link
Member

Looking at this now @Cronus1007 , apologies for the delay. Can you rebase to resolve conflicts in packages/ui-markdown-editor/package.json?

@Cronus1007
Copy link
Contributor Author

@irmerk I have resolved merge conflicts.

---
The functions in `clauseProps` like `CLAUSE_DELETE_FUNCTION`,`CLAUSE_EDIT_FUNCTION` and `CLAUSE_TEST_FUNCTION` are not implemented by default because they typically require interaction with other components or the persistence layer of the embedding application.
These functions will be supplied by the user of the component.
For more information in realtion to the `props` , `values` and `functionality` of Contract Editor visit [README.md](./src/ContractEditor/README.md)
Copy link
Member

Choose a reason for hiding this comment

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

One last change, I think we should remove this last line since that README file no longer exists.

Copy link
Member

Choose a reason for hiding this comment

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

Also keep the --- horizontal rule just after this section you added.

@jolanglinais jolanglinais linked an issue Apr 6, 2021 that may be closed by this pull request
@jolanglinais
Copy link
Member

Still unsure about the huge addition to package-lock when dependencies are being removed. Please rebase to one commit.

@Cronus1007 Cronus1007 force-pushed the issue- branch 2 times, most recently from 318fb19 to e3831cf Compare April 6, 2021 17:40
@Cronus1007 Cronus1007 closed this May 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Package: Markdown Editor ⌨️ Type: Dependencies 🏠 Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docs for Delete Edit and Test Icons Dependencies Cleanup
4 participants