Skip to content

Conversation

@mistercrunch
Copy link
Member

Attempt at bumping react-select to rc10 to remove React warning messages. Also cleaning up the copy-pasta of select.less.

Somehow the css is a little off and can't figure it out. I think it has to do with this hack here:
https://github.com/apache/incubator-superset/blob/master/superset/assets/stylesheets/superset.less#L344

screen shot 2017-10-27 at 12 28 28 am

@graceguo-supercat can you help getting this through?

@coveralls
Copy link

coveralls commented Oct 27, 2017

Coverage Status

Coverage remained the same at 71.271% when pulling 5c54230 on mistercrunch:bump-react-select into cbd0107 on apache:master.

@graceguo-supercat
Copy link

LGTM

@graceguo-supercat graceguo-supercat merged commit abfa034 into apache:master Nov 1, 2017
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this pull request Nov 1, 2017
graceguo-supercat pushed a commit that referenced this pull request Nov 2, 2017
michellethomas pushed a commit to michellethomas/panoramix that referenced this pull request May 24, 2018
michellethomas pushed a commit to michellethomas/panoramix that referenced this pull request May 24, 2018
wenchma pushed a commit to wenchma/incubator-superset that referenced this pull request Nov 16, 2018
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.20.5 First shipped in 0.20.5 labels Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.20.5 First shipped in 0.20.5

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants