Open HarikrishnanBalagopal opened 3 years ago
Avoids chance for ambiguity here: https://github.com/konveyor/move2kube/blob/main/internal/move2kube/planner.go#L206-L216 once this is fixed.
We should probably add another text named say, display text instead of changing the current behavior, so more robust display engines like cli and web can use them. For things like settings, we can retain current behavior.
Indices need to rely on orders which will be a challenge.
Description
During
Select
andMultiSelect
type QA we present some options to the user and they select one or more of them. To make it simple for the user we simplify the options (ex: make absolute paths into relative paths) However this can cause issues in certain corner cases where the options can become ambiguous.It also limits us by not being able to show extra metadata that would be useful for the user. Ex: Display translation type next to the service name.
Proposed Fix
Use indices instead of strings. Ex:
This allows us to display something nice to the user while using a different thing internally.