Skip to content

OnboardingTopBar: remove unneeded prop type#1136

Merged
sohkai merged 2 commits intoaragon:masterfrom
adekbadek:prop-types-fix
Oct 23, 2019
Merged

OnboardingTopBar: remove unneeded prop type#1136
sohkai merged 2 commits intoaragon:masterfrom
adekbadek:prop-types-fix

Conversation

@adekbadek
Copy link
Copy Markdown
Contributor

After the last update (bbb1038) the prop
and imports were not removed.

After the last update (bbb1038) the prop
and imports were not removed.
@vercel vercel bot temporarily deployed to staging October 18, 2019 15:12 Inactive
Copy link
Copy Markdown
Contributor

@bpierre bpierre left a comment

Choose a reason for hiding this comment

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

🙈

@adekbadek
Copy link
Copy Markdown
Contributor Author

@bpierre I just noticed ErrorScreen also has unused imports after same changes ;)

@vercel vercel bot temporarily deployed to staging October 23, 2019 16:04 Inactive
@sohkai sohkai merged commit e5cbf71 into aragon:master Oct 23, 2019
chadoh added a commit to AutarkLabs/aragon that referenced this pull request Oct 23, 2019
* 'master' of github.com:aragon/aragon: (23 commits)
  Templates: allow disabled, 'coming soon' templates to be opened (aragon#1141)
  OnboardingTopBar, ErrorScreen: remove unneeded prop types (aragon#1136)
  Error screen: various fixes (aragon#1132)
  UI: Replace Checkbox with Switch where applicable (aragon#1135)
  Local settings: fix two setters (aragon#1129)
  Build: update to core-js@3 (aragon#1126)
  Configuration: move prefix from REACT_APP to ARAGON (aragon#1122)
  Add and configure Jest (aragon#1123)
  AppIFrame: fix iframe not being mounted in IE (aragon#1125)
  0.8.2 (aragon#1120)
  Onboarding: minor fixes and improvements (aragon#1117)
  Permissions: refactor to use SearchInput (aragon#1100)
  Templates: add Fundraising (aragon#1092)
  Custom labels: update search and DataView (aragon#1077)
  RepoBadge: update style and fix how it loads repo content (aragon#1118)
  Onboarding: minor fixes (aragon#1119)
  Onboarding: clamp value of Voting's support and quorum to their max values (aragon#1114)
  Onboarding welcome: UI fixes (aragon#1110)
  Onboarding: remove information about waiting for transactions t… (aragon#1113)
  docs: add geleeroyale as a contributor (aragon#1112)
  ...
chadoh added a commit to AutarkLabs/aragon that referenced this pull request Oct 23, 2019
* master: (24 commits)
  add framework for conditionally rendering profiles
  Templates: allow disabled, 'coming soon' templates to be opened (aragon#1141)
  OnboardingTopBar, ErrorScreen: remove unneeded prop types (aragon#1136)
  Error screen: various fixes (aragon#1132)
  UI: Replace Checkbox with Switch where applicable (aragon#1135)
  Local settings: fix two setters (aragon#1129)
  Build: update to core-js@3 (aragon#1126)
  Configuration: move prefix from REACT_APP to ARAGON (aragon#1122)
  Add and configure Jest (aragon#1123)
  AppIFrame: fix iframe not being mounted in IE (aragon#1125)
  0.8.2 (aragon#1120)
  Onboarding: minor fixes and improvements (aragon#1117)
  Permissions: refactor to use SearchInput (aragon#1100)
  Templates: add Fundraising (aragon#1092)
  Custom labels: update search and DataView (aragon#1077)
  RepoBadge: update style and fix how it loads repo content (aragon#1118)
  Onboarding: minor fixes (aragon#1119)
  Onboarding: clamp value of Voting's support and quorum to their max values (aragon#1114)
  Onboarding welcome: UI fixes (aragon#1110)
  Onboarding: remove information about waiting for transactions t… (aragon#1113)
  ...
@sohkai
Copy link
Copy Markdown
Contributor

sohkai commented Oct 24, 2019

@all-contributors please add @adekbadek for code!

@allcontributors
Copy link
Copy Markdown
Contributor

@sohkai

I've put up a pull request to add @adekbadek! 🎉

@aragon aragon deleted a comment from allcontributors bot Oct 24, 2019
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.

3 participants