enso-org / enso

Enso Analytics is a self-service data prep and analysis platform designed for data teams.
https://ensoanalytics.com
Apache License 2.0
7.38k stars 323 forks source link

Merge `URI` with `URI_With_Query` #8544

Closed radeusgd closed 11 months ago

radeusgd commented 11 months ago

After some followup work and discussion, we conjecture that it may be cleaner to merge the utility of URI_With_Query directly into the URI, to have more uniform handling.

The APIs should be merged, we may want to add following additional methods:

enso-bot[bot] commented 11 months ago

Radosław Waśko reports a new STANDUP for the provided date (2023-12-15):

Progress: Getting all pending PRs forward - fixing some tests, a few already got merged. Work on merging URI with URI_With_Query. It should be finished by 2023-12-20.

Next Day: Next day I will be working on the same task. Continue - creating a common way to create a URI from a 'schematic' that may contain secrets inside of query or user_info. Addressing PR comments.

enso-bot[bot] commented 11 months ago

Radosław Waśko reports a new STANDUP for yesterday (2023-12-18):

Progress: Removed Apache dependency. Created a common URI API. Working on making tests pass It should be finished by 2023-12-20.

Next Day: Next day I will be working on the same task. Fix remaining test failures

enso-bot[bot] commented 11 months ago

Radosław Waśko reports a new STANDUP for today (2023-12-19):

Progress: Pulling the PRs thru reviews and CI, merged a lot. Prepared the URI merge PR for review. Discussion with Jaroslav on next steps on types. It should be finished by 2023-12-20.

Next Day: Next day I will be working on the #8549 task. Pick up some next tasks.