fix(Explore): Undefined owners

[geido]

SUMMARY

This PR fixes the following issues:

• Undefined owners in the Datasource modal
• Impossible to save with the same owners

The problem arose from the fact that the data related to the owners is very inconsistent and comes in three forms:

• On load the data is of type [{id: number; first_name: string; last_name: string;}]
• On selection from the dropdown the data is of type [{value: number; label: string;}]
• On save the data is of type number[]

This PR checks for the different data types to make it work. However, a deeper refactor is required to make the data consistent in both the frontend and the backend

Follow-up of #17063

BEFORE

before.mp4

AFTER

after.mp4

TESTING INSTRUCTIONS

1. Open Explore
2. Click on "Edit dataset"
3. Select an owner and save
4. Re-open the modal and check the owners

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #17167 (1e29966) into master (e32acd2) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master   #17167   +/-   ##
=======================================
  Coverage   76.91%   76.91%           
=======================================
  Files        1039     1039           
  Lines       55559    55561    +2     
  Branches     7567     7570    +3     
=======================================
+ Hits        42733    42735    +2     
  Misses      12576    12576           
  Partials      250      250           

Flag	Coverage Δ
javascript	70.93% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...end/src/components/Datasource/DatasourceEditor.jsx	72.10% <100.00%> (+0.19%)	⬆️
...tend/src/components/Datasource/DatasourceModal.tsx	75.38% <100.00%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e32acd2...1e29966. Read the comment docs.

[michael-s-molina]

/testenv up

[github-actions]

@michael-s-molina Ephemeral environment spinning up at http://54.218.248.133:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[michael-s-molina]

LGTM.

[rusackas]

Looks like it works in testing!

[rusackas]

It would probably be a good thing to revisit this one day to make things more aligned/consistent between frontend and backend and simplify some of this logic.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.

[rosemarie-chiu]

🏷 2021.40