hi folks I just noticed something about the KUG map with <https github com JetBrains kotlin web site commit 7753893f1bdae16c3ef1587c609ad6400ac09db6 diff 7d96afb26d631db2ec0ae1b4e3ee00544d880bb42255e5ba249ed53d4c5365c1|this change> we don t require position for KUGs because the virtual user group doesn t have one But then there s no mechanism to ensure that other physical user groups have them see <https github com JetBrains kotlin web site commit 41ea9340b2bdffe5c0c2a1cbd173c5433d741c50 diff fdf1e3810128bfbaa80ebcd9b93d543aaf795f875dd8567ffe571d41255a3897|last change> which added two UGs without position My idea for improvement restore `position` being mandatory but add a condition for the virtual KUG that it s not placed on the map VKUG could have some dummy position like 0 0 0 0 WDYT

hi folks! I just noticed something about the KUG map: with this change, we don't require position for KUGs because the virtual user group doesn't have one. But then there's no mechanism to ensure that other physical user groups have them, see last change which added two UGs without position. My idea for improvement: restore

position

being mandatory, but add a condition for the virtual KUG that it's not placed on the map. VKUG could have some dummy position, like (0.0, 0.0). WDYT?

CC @Alina Dolgikh [JB]

I created KT-49574: New user groups are absent on the map for the sake of proper tracking

:thank-you: 1

Alina Dolgikh [JB]

11/05/2021, 1:16 PM

Hi Piotr. I see the issue.

My idea for improvement: restore
position
being mandatory, but add a condition for the virtual KUG that it’s not placed on the map. VKUG could have some dummy position, like (0.0, 0.0). WDYT?

sounds good to me. WDYT @Nikolay Pachkov?

Nikolay Pachkov

11/08/2021, 10:39 AM

Hi Piotr! That's a good idea, thanks a lot!

💪 1

👍 1

4 Views

#docs-revamped Join Slack