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

Add styleguide to workspace #1243

Merged
merged 1 commit into from
Jun 27, 2019
Merged

Conversation

rupurt
Copy link
Contributor

@rupurt rupurt commented May 20, 2019

  • Move OperatorUI theme
  • Move OperatorUI TimeAgo, Tooltip, KeyValueList, Logo & Image
  • Move OperatorUI shared cards
  • Added test:nocache task to force a recompile in jest

Typescript support is getting worked on in a separate branch as it raises 100's errors across the project and will result in an even larger diff. I've converted the old tsx components to jsx for this PR but will restore them to tsx when the remaining errors are fixed`

@rupurt rupurt force-pushed the chores/add-styleguide-to-workspace branch 2 times, most recently from a5f54d4 to c7b28e6 Compare May 23, 2019 23:40
@rupurt rupurt changed the title Add styleguide to workspace [WIP] Add styleguide to workspace May 23, 2019
@rupurt rupurt force-pushed the chores/add-styleguide-to-workspace branch 2 times, most recently from 0fb5428 to d7cf8de Compare May 24, 2019 00:00
@se3000 se3000 changed the base branch from master to develop June 9, 2019 19:38
@rupurt rupurt force-pushed the chores/add-styleguide-to-workspace branch 8 times, most recently from 04cbea6 to 08657a3 Compare June 25, 2019 23:03
@rupurt rupurt changed the title [WIP] Add styleguide to workspace Add styleguide to workspace Jun 26, 2019
- Move OperatorUI theme
- Move OperatorUI TimeAgo, Tooltip, KeyValueList, Logo & Image
- Move OperatorUI shared cards
@rupurt rupurt force-pushed the chores/add-styleguide-to-workspace branch from 08657a3 to de8d4e2 Compare June 27, 2019 15:44
Copy link
Contributor

@se3000 se3000 left a comment

Choose a reason for hiding this comment

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

See some stuff moved into the styleguide also getting converted from TS to JS. There's no reason we can't switch that back in the future, right? I'm assuming it is just extra complication.

@rupurt
Copy link
Contributor Author

rupurt commented Jun 27, 2019

@se3000 yeah, I'm working on the Typescript support. Adding Typescript support to the styleguide in workspaces has strict checking. This highlighted that there are a bunch of annotations that are missing and are needed to get explorer + styleguide + operator_ui in sync. Actively working on that in this branch

https://github.com/smartcontractkit/chainlink/tree/chores/fix-typescript-support-with-styleguide

@rupurt rupurt merged commit 314f68a into develop Jun 27, 2019
@rupurt rupurt deleted the chores/add-styleguide-to-workspace branch June 27, 2019 16:43
asoliman92 pushed a commit that referenced this pull request Aug 27, 2024
## Motivation
`NonceManager` and `MultiAggregateRateLimiter` contracts were missing
`typeAndVersion`.

## Solution
Add `ITypeAndVersion` inheritance to both contracts.
asoliman92 pushed a commit that referenced this pull request Aug 28, 2024
## Motivation
`NonceManager` and `MultiAggregateRateLimiter` contracts were missing
`typeAndVersion`.

## Solution
Add `ITypeAndVersion` inheritance to both contracts.
asoliman92 pushed a commit that referenced this pull request Aug 28, 2024
## Motivation
`NonceManager` and `MultiAggregateRateLimiter` contracts were missing
`typeAndVersion`.

## Solution
Add `ITypeAndVersion` inheritance to both contracts.
asoliman92 pushed a commit that referenced this pull request Aug 28, 2024
## Motivation
`NonceManager` and `MultiAggregateRateLimiter` contracts were missing
`typeAndVersion`.

## Solution
Add `ITypeAndVersion` inheritance to both contracts.
asoliman92 pushed a commit that referenced this pull request Aug 28, 2024
## Motivation
`NonceManager` and `MultiAggregateRateLimiter` contracts were missing
`typeAndVersion`.

## Solution
Add `ITypeAndVersion` inheritance to both contracts.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants