redash icon indicating copy to clipboard operation
redash copied to clipboard

[Parse dataSourceIds from first parameter]

Open ilgwonPark2 opened this issue 3 years ago • 0 comments

  • localStorage.getItem("lastSelectedDataSourceId") returns string value
  • Since this structure is not based on TS, I suggest parse three items in dataSourceIds. Or you can parse just localStorage.getItem("lastSelectedDataSourceId")

What type of PR is this?

  • [ ] Refactor
  • [ ] Feature
  • [V] Bug Fix
  • [ ] New Query Runner (Data Source)
  • [ ] New Alert Destination
  • [ ] Other

Description

How is this tested?

  • [ ] Unit tests (pytest, jest)
  • [ ] E2E Tests (Cypress)
  • [V ] Manually
  • [ ] N/A

manually tested, if redash client data source remembers correctly and parse local storage data

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

ilgwonPark2 avatar Jul 07 '22 04:07 ilgwonPark2