search

teragrep / ajs_01

User interface for Teragrep
Apache License 2.0
0 stars 0 forks source link

You can add multiple duplicated fields in graph values section breaking the UI #172

Closed kortemik closed 11 months ago

kortemik commented 1 year ago

ID <274> Reporter <@StrongestNumber9>

Peek_2022-11-07_10-00.webm

BVVLD commented 1 year ago

The feature is implemented using angular-dragdrop library. It has deduplication, but in the values section the data is altered before adding to the section, so it processed as new object. Currently we have a solution to remove the duplicate fields, however, there is a potential if user wants to add same field with different operator (Sum, avg, etc) it might be a bit problematic, since it will require first to change previous field of same name into another operator and then add it again, with default SUM operator. Needs redesign of the UX. The issue might also be resolved with replacement of the lib.

BVVLD commented 11 months ago

Should be fixed in new drag&drop component