-
Notifications
You must be signed in to change notification settings - Fork 13
Conversation
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 we need to update run instructions here(both local and remote in the docker)?
Also, I see a bunch of files with js and jsx extensions. Do we really need to preserve these extensions?
src/theme.tsx
Outdated
// TO-DO delete theme dont have this values | ||
// mediumLight: '#ba68c8', | ||
// white: '#f5f5f5' |
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.
why not delete it right now?
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.
yes we can!
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.
fixed
src/utils/index.ts
Outdated
@@ -3,7 +3,7 @@ | |||
* @param {*} escrowFactories Array of escrow Factories | |||
* @returns the total count of factory | |||
*/ | |||
export const countEscrowFactory = (escrowFactories) => { | |||
export const countEscrowFactory = (escrowFactories: Array<any>) => { |
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.
Let's add a type here. Would be easier to understand, what this function is doing actually
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.
added
What about running instructions(both local and remote)? Are they the same as were before? |
started typescript migration