- Issue created by @eme
- First commit to issue fork.
- Merge request !230Refactor selected_text handling to avoid URL length limitations → (Merged) created by k_a_l
- 🇺🇸United States k_a_l
After troubleshooting this, it's more so an issue with sending the selected text to the jQuery UI dialog, which is done as a query string. The AI request is already being sent as a POST request. I had the same issue, so I decided to refactor how the data is sent by using a JavaScript object instead of query parameters.
- 🇬🇧United Kingdom scott_euser
Side note, this makes me think we should remove the SKIP_ESLINT and sort that. Its hard to review the code for this as its so much coding standard change
- Status changed to Needs review
about 1 month ago 9:36pm 16 January 2025 - 🇺🇸United States k_a_l
Hi Scott, understood. It's likely my IDE was making additional improvements. I synced with 1.0.x and removed some of the minor updates to improve diff readability. Thanks
- 🇺🇸United States thejimbirch Cape Cod, Massachusetts
Closed my issue as a duplicate of this. The solution looks promising.
- 🇩🇪Germany marcus_johansson
Tested and working - looks good. Merging into 1.0.0-dev. Thank you k_a_l!
Automatically closed - issue fixed for 2 weeks with no activity.