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

Developer UI: Header, connect a provider #134

Merged
merged 59 commits into from
Apr 22, 2021

Conversation

mmv08
Copy link
Member

@mmv08 mmv08 commented Apr 15, 2021

This PR:

  • Bootstraps safe-dev-ui project
  • Adds a header

@github-actions
Copy link
Contributor

CLA Assistant Lite All Contributors have signed the CLA.

@mmv08 mmv08 closed this Apr 15, 2021
@mmv08 mmv08 deleted the feature/dev-ui-welcome-page branch April 15, 2021 10:56
@github-actions github-actions bot locked and limited conversation to collaborators Apr 15, 2021
@mmv08 mmv08 restored the feature/dev-ui-welcome-page branch April 15, 2021 11:04
@mmv08 mmv08 reopened this Apr 15, 2021
@mmv08 mmv08 marked this pull request as ready for review April 15, 2021 11:10
@mmv08 mmv08 requested review from rmeissner and germartinez and removed request for rmeissner April 15, 2021 11:12
@mmv08
Copy link
Member Author

mmv08 commented Apr 15, 2021

Eslint/Cla actions are failing because the PR was closed before. You can find ESLint annotation on the PR with all changes: #134

packages/safe-apps-developer-ui/src/utils/strings.ts Outdated Show resolved Hide resolved
export function getLowerCase(value: string): string {
if (value) {
return value.toLowerCase();
}
return value;
}

export function convertSafeTxToEthersTx(tx: TxServiceModel): TransactionResponse {
Copy link
Member

Choose a reason for hiding this comment

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

Is this used somewhere?

Copy link
Member Author

Choose a reason for hiding this comment

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

no idea what it's doing here

class Provider extends React.Component<any> {
myRef: any;

constructor(props: any) {
Copy link
Member

Choose a reason for hiding this comment

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

Could you specify the type of the props?

@mmv08 mmv08 merged commit d2e86c6 into development Apr 22, 2021
@mmv08 mmv08 deleted the feature/dev-ui-welcome-page branch April 22, 2021 10:04
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.

2 participants