dfpc-coe / CloudTAK

Full Featured in-browser TAK Client powered by AWS
GNU Affero General Public License v3.0
16 stars 3 forks source link

Update connection-channels.ts #392

Closed sahil270803 closed 1 month ago

sahil270803 commented 1 month ago

added a warning check in the /connection/:connectionid/channel GET endpoint that monitors when users select multiple channels for data sync. Specifically, we added a simple condition that checks if list.data.length > 1 and if true, adds a warning message to the response object. This fulfills issue #387's requirement to warn users when they select more than one channel in their Data Sync channel list.

ingalls commented 1 month ago

Hey @sahil270803,

The context of the #387 is that of the Mission Sync UI and not of the Connection API. The change will also need to be implemented on the front-end as a warning before creation and not as a backhand property.

If you are interested in more context or have questions feel free to comment on #387 for clarification or for guidance on where these changes would need to be made. As this PR does not fulfill the desired actions outlined in the issue I am going to close.