Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

Rif ui extraction #48

Merged
merged 5 commits into from
Apr 30, 2020
Merged

Rif ui extraction #48

merged 5 commits into from
Apr 30, 2020

Conversation

itofarina
Copy link
Contributor

@itofarina itofarina commented Apr 28, 2020

This PR removes the dependency on the rifui folder and adds the dependency to the library that now lives in RIF-UI repository.
It also has the changes that the folder needed in order to be used as a library so we can review those changes here :).

Alert

This PR has a dependency on the new library, pointing to the feature/extraction branch, please make sure it's still alive when reviewing or update the reference to master in the package.json file, just like this:

...,
"dependencies": {
...
    "@rsksmart/rif-ui": "github:rsksmart/rif-ui#feature/extraction",
...
},

Known issues

  • More like an issue to be solved in RIF-UI but we don't want to import files specifying the dist path.
  • Similar as above, it would be ideal to import for example the theme from @rsksmart/rif-ui/theme.

@julianmrodri julianmrodri linked an issue Apr 29, 2020 that may be closed by this pull request
@jurajpiar
Copy link
Member

jurajpiar commented Apr 30, 2020

Cannot successfully run it. The web3 account seems to be still undefined after login in.

@itofarina itofarina force-pushed the feature/rifUiExtraction branch from cb941ba to ba1869d Compare April 30, 2020 16:40
package.json Outdated Show resolved Hide resolved
src/components/atoms/Icon.tsx Show resolved Hide resolved
src/components/templates/MarketPageTemplate.tsx Outdated Show resolved Hide resolved
@itofarina itofarina merged commit 6b96136 into master Apr 30, 2020
@itofarina itofarina deleted the feature/rifUiExtraction branch April 30, 2020 18:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

RIF-UI library
4 participants