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

Handle ENTER key correctly in trigger form and allow manual JSON #42525

Conversation

jscheffl
Copy link
Contributor

Follow-up of #42487

related: #42157

Unfortunately a bit more JavaScript is needed but this enables:

  • Handling of press of ENTER in input fields and ensure that JSON form is submitted and correct value is sent
  • Handling of manual changes of JSON dict (fix before: was always over-writing JSON)

@dannyl1u - can you confirm from your side?

@jscheffl jscheffl added the area:UI Related to UI/UX. For Frontend Developers. label Sep 26, 2024
@jscheffl jscheffl added this to the Airflow 2.10.3 milestone Sep 26, 2024
@boring-cyborg boring-cyborg bot added the area:webserver Webserver related Issues label Sep 26, 2024
@jscheffl jscheffl added type:bug-fix Changelog: Bug Fixes and removed area:webserver Webserver related Issues labels Sep 26, 2024
@dannyl1u
Copy link
Contributor

Follow-up of #42487

related: #42157

Unfortunately a bit more JavaScript is needed but this enables:

  • Handling of press of ENTER in input fields and ensure that JSON form is submitted and correct value is sent
  • Handling of manual changes of JSON dict (fix before: was always over-writing JSON)

@dannyl1u - can you confirm from your side?

Taking a look

@dannyl1u
Copy link
Contributor

@jscheffl tested your branch locally, works good for me. Thank you!

Copy link
Member

@pierrejeambrun pierrejeambrun left a comment

Choose a reason for hiding this comment

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

Thanks for catching that early.

Tested, working as expected. Code looks good. 👍

@pierrejeambrun pierrejeambrun merged commit 3ef29a6 into apache:main Sep 27, 2024
55 checks passed
pierrejeambrun pushed a commit to astronomer/airflow that referenced this pull request Sep 27, 2024
@pierrejeambrun
Copy link
Member

Thanks, cherrypick PR is here #42535

shahar1 pushed a commit that referenced this pull request Sep 27, 2024
joaopamaral pushed a commit to joaopamaral/airflow that referenced this pull request Oct 21, 2024
utkarsharma2 pushed a commit that referenced this pull request Oct 23, 2024
utkarsharma2 pushed a commit that referenced this pull request Oct 24, 2024
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:UI Related to UI/UX. For Frontend Developers. type:bug-fix Changelog: Bug Fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants