Skip to content

fix(KeyValue Editor): fix key value focus issue and handle updating params from url#7780

Merged
gatzjames merged 3 commits intoKong:developfrom
gatzjames:feature/ins-4215-fix-key-value-first-input
Jul 30, 2024
Merged

fix(KeyValue Editor): fix key value focus issue and handle updating params from url#7780
gatzjames merged 3 commits intoKong:developfrom
gatzjames:feature/ins-4215-fix-key-value-first-input

Conversation

@gatzjames
Copy link
Contributor

Highlights:

  • Fixes an issue where typing on the first empty item would lose focus
  • Fixes an issue where importing params from the url would not work
  • Fixes an issue where having a disabled item and then enabling it wouldn't allow for it to be editable

@gatzjames gatzjames self-assigned this Jul 30, 2024
@gatzjames gatzjames requested a review from a team July 30, 2024 12:41
@gatzjames gatzjames force-pushed the feature/ins-4215-fix-key-value-first-input branch from 3fc356a to cbcff58 Compare July 30, 2024 12:42
@gatzjames gatzjames enabled auto-merge (squash) July 30, 2024 12:42
@gatzjames gatzjames force-pushed the feature/ins-4215-fix-key-value-first-input branch from d5d68f6 to 447fb71 Compare July 30, 2024 13:23
@gatzjames gatzjames merged commit a65a875 into Kong:develop Jul 30, 2024
@gatzjames gatzjames deleted the feature/ins-4215-fix-key-value-first-input branch July 30, 2024 13:34
@sentry
Copy link

sentry bot commented Aug 1, 2024

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

  • ‼️ Error: Expected fetch controller: :r55: app:///index.html View Issue
  • ‼️ Error: Expected fetch controller: :r2s: app:///index.html View Issue
  • ‼️ Error: Expected fetch controller: :r5v: app:///index.html View Issue
  • ‼️ Error: Expected fetch controller: :r7c: app:///index.html View Issue
  • ‼️ Error: Expected fetch controller: :r4d: app:///index.html View Issue

Did you find this useful? React with a 👍 or 👎

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.

2 participants