* refactor: 💡 allow for graceful error handling when connecting
to a target where host-sets cannot be fetched
* refactor: 💡 update for loop to for...of loop
* Move fetching host from target to session domain
* Decouple getting target host-sets and getting hosts from the host-sets, now this code is extracted in 2 methods. Now if the user does not have permissions to fetch host or host-sets we do not show an error and we allow for quick-connect approach
* Fix a check within getHostsFromHostSets
* Change a for loop for Promise.all within getTargetHostSets
* Change a for loop for Promise.All within getHostsFromHostSets
* Delete unnecessary condition
* Delete unnecessary condition
* chore: 🤖 remove host sources from target mirage factory
by default it uses address when using 'withAssociations'
* test: 💍 update target address tests to use 'withOneHost'
* test: 💍 remove connect tests from scopes acceptance tests
* refactor: 💡 remove can initiate connect target ability
and associated code and tests
* refactor: 💡 update target model afterModel hook, and preconnect
* refactor: 💡 revert 'withAssociations' mirage change
* test: 💍 add tests for permissions issues for host-sets and host
* test: 💍 visiting session without permissions to read hosts
---------
Co-authored-by: cameronperera <cameron.perera@hashicorp.com>
* fix: 🐛 host modal renders whenever there is an error
✅ Closes: ICU-11231
* test: 💍 add test case to make sure fix is working properly
* fix: 🐛 toggle flag back to false and remove @tracked
* fix: 🐛 host modal should also close after host connection error
* refactor: 💡 change toggle to toggleModal
* test: 💍 rename toggle in test cases
* test: 💍 update acceptance tests in desktop UI
✅ Closes: ICU-11130
* test: 💍 move tests to target-test.js
* test: 💍 add another test case for targets list view
* fix: 🐛 Removing status badging from desktop client
Removed status badges for `targets` and `sessions` in desktop client
app.
✅ Closes: ICU-9790
* refactor: 💡 Bring back status badging for session
Session status badging can stay due to session can only be canceled by
the user in desktop client and does give an immediate status to the
session.
✅ Closes: ICU-9790
* refactor: 💡 merge changes
Additional cleanup: remove unused async calls, separate session and
associated manager models, & attempt encoding origin url as part of
sanitization for cli command
Zhihe Li
Carlos Alcaide Corvoandcameronperera
Devon Anderson
Randall Morey
hashicorp-copywrite[bot]andCameron Perera
Susmitha Girumala
Carlos Alcaide