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

refactor(cicero-ui): remove prop dependency - I70 #107

Merged

Conversation

jolanglinais
Copy link
Member

Closes #70

Reduce prop reliance for cicero-ui

Changes

  • Remove styling props
  • Sync class names with Library component

Flags

  • Did not address tests

Related Issues

Author Checklist

  • 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

Signed-off-by: irmerk <jolenelanglinais@gmail.com>
Signed-off-by: irmerk <jolenelanglinais@gmail.com>
Signed-off-by: irmerk <jolenelanglinais@gmail.com>
Signed-off-by: irmerk <jolenelanglinais@gmail.com>
@DianaLease
Copy link
Member

Sooo much cleaner 🙏

Signed-off-by: irmerk <jolenelanglinais@gmail.com>
@jolanglinais
Copy link
Member Author

Just make sure e68ad08 is all good.

Signed-off-by: irmerk <jolenelanglinais@gmail.com>
@jolanglinais jolanglinais merged commit e038e0c into accordproject:master Jun 3, 2020
@jolanglinais jolanglinais deleted the irmerk/i70/prop-reliance branch June 3, 2020 12:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reduce Prop Reliance
2 participants