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

Fix request new connector button #16934

Merged
merged 3 commits into from
Sep 21, 2022

Conversation

krishnaglick
Copy link
Contributor

@krishnaglick krishnaglick commented Sep 20, 2022

What

The Request New Connector button was not taking clicks. Likely due to the react-select upgrade.
Resolves https://github.com/airbytehq/oncall/issues/630

How

Setting position: relative; let it be interacted with again.

🚨 User Impact 🚨

This should not affect anything besides the problem

Scope Creep?

There are two changes outside the strict scope of the PR:

  1. Typing an any
  2. The problem component had a reference to greyColro0 which does not exist. I fixed it there and in one other place.

@krishnaglick krishnaglick requested a review from a team as a code owner September 20, 2022 16:53
@github-actions github-actions bot added area/platform issues related to the platform area/frontend Related to the Airbyte webapp labels Sep 20, 2022
@krishnaglick
Copy link
Contributor Author

I tested this with #16937 merged in and did not see any regression.

Copy link
Contributor

@teallarson teallarson left a comment

Choose a reason for hiding this comment

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

Tested locally. Modal appears on click, submitting it fires a segment call.

@krishnaglick krishnaglick merged commit 83f5d9a into master Sep 21, 2022
@krishnaglick krishnaglick deleted the kc/fix-request-new-connector-button branch September 21, 2022 13:55
robbinhan pushed a commit to robbinhan/airbyte that referenced this pull request Sep 29, 2022
* Combine div's, set positive to relative

* Type args

* Fix theme reference
jhammarstedt pushed a commit to jhammarstedt/airbyte that referenced this pull request Oct 31, 2022
* Combine div's, set positive to relative

* Type args

* Fix theme reference
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/frontend Related to the Airbyte webapp area/platform issues related to the platform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants