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

Stream Details panel - update predefined cursor and primary fields behavior #20227

Closed
dizel852 opened this issue Dec 8, 2022 · 1 comment · Fixed by #20366
Closed

Stream Details panel - update predefined cursor and primary fields behavior #20227

dizel852 opened this issue Dec 8, 2022 · 1 comment · Fixed by #20366
Assignees
Labels
area/frontend Related to the Airbyte webapp team/platform-move

Comments

@dizel852
Copy link
Contributor

dizel852 commented Dec 8, 2022

Tell us about the problem you're trying to solve

At the moment, we don't show the selected cursor field in the new details panel(#19219) if it was predefined and we can't change it.

Describe the solution you’d like

After discussing possible UX solutions with @edmundito and @Upmitt we agreed:

  • Fill in the Cursor column with disabled <RadioButtons>'s
  • Select the cursor field which is predefined
    • Fill in the Primary Key column with disabled <CheckBox>'s
  • Select the primary key field which is predefined

Since we don't have a "selected disabled" state in the current implementation of `', so need to update it first:
#20089

Not relevant for <RadioButton>, the desired state is supported.
But still relevant for <CheckBox> component: #20246

Describe the alternative you’ve considered or used

A clear and concise description of any alternative solutions or features you've considered or are using today.

@octavia-squidington-iv
Copy link

cc @airbytehq/frontend

@dizel852 dizel852 self-assigned this Dec 8, 2022
@dizel852 dizel852 changed the title [BLOCKED]Stream Details panel - update predefined cursor field behavior Stream Details panel - update predefined cursor field behavior Dec 8, 2022
@dizel852 dizel852 closed this as completed Dec 8, 2022
@dizel852 dizel852 changed the title Stream Details panel - update predefined cursor field behavior Stream Details panel - update predefined cursor and primary fields behavior Dec 8, 2022
@dizel852 dizel852 reopened this Dec 8, 2022
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 team/platform-move
Projects
None yet
2 participants