Skip to content

Conversation

dididy
Copy link
Contributor

@dididy dididy commented Aug 31, 2025

What is this PR for?

Description:
Changing the interpreter for a paragraph did not update the binding list area when clicking the Interpreter Binding button in the action bar. Although the server had the correct values, the client did not call the update function.

[Appropriate action - Classic UI]

2025-08-28.2.24.41.mov

[AS-IS]

2025-08-28.2.25.19.mov

[TO-BE]

2025-08-31.9.31.37.mov

What type of PR is it?

Bug Fix

Todos

What is the Jira issue?

How should this be tested?

Screenshots (if appropriate)

Questions:

  • Does the license files need to update? N
  • Is there breaking changes for older versions? N
  • Does this needs documentation? N

Copy link
Contributor

@tbonelee tbonelee left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ParkGyeongTae ParkGyeongTae left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@tbonelee tbonelee merged commit a0ef5b8 into apache:master Sep 2, 2025
20 of 30 checks passed
tbonelee pushed a commit that referenced this pull request Sep 2, 2025
…n reopening the binding settings

### What is this PR for?
**Description:**
Changing the interpreter for a paragraph did not update the binding list area when clicking the `Interpreter Binding` button in the **action bar**. Although the server had the correct values, the client did not call the update function.

**[Appropriate action - Classic UI]**

https://github.com/user-attachments/assets/f2bbb602-056f-4f09-8015-e0268233f246

**[AS-IS]**

https://github.com/user-attachments/assets/38b41567-eccd-47b6-942d-afa603d86dff

**[TO-BE]**

https://github.com/user-attachments/assets/b9959eb8-5408-49fb-a9e5-15a725ca048a

### What type of PR is it?
Bug Fix

### Todos

### What is the Jira issue?
* [[ZEPPELIN-6304](https://issues.apache.org/jira/browse/ZEPPELIN-6304)]

### How should this be tested?

### Screenshots (if appropriate)

### Questions:
* Does the license files need to update? N
* Is there breaking changes for older versions? N
* Does this needs documentation? N

Closes #5061 from dididy/fix/ZEPPELIN-6304.

Signed-off-by: ChanHo Lee <[email protected]>
(cherry picked from commit a0ef5b8)
Signed-off-by: ChanHo Lee <[email protected]>
@tbonelee
Copy link
Contributor

tbonelee commented Sep 2, 2025

Thanks! I've merged this into master and branch-0.12

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