Skip to content

BYNT-1383: Set default for radius#128

Merged
tarekio merged 3 commits intomainfrom
bynt-1383-map-radius-fix
Jul 9, 2025
Merged

BYNT-1383: Set default for radius#128
tarekio merged 3 commits intomainfrom
bynt-1383-map-radius-fix

Conversation

@cango91
Copy link
Contributor

@cango91 cango91 commented Jul 4, 2025

Jira Issue

  1. BYNT-1383

Description

Previously, the GeoMap component always included radius in emitted values even when radiusControls was disabled, causing unwanted diffs in System Settings when only updating map center coordinates. Updates the component to emit radius value only if radius controls are enabled.
see review

Checklist

  • Tests added/updated
  • Documentation updated (if needed)
  • New strings prepared for translations

API Changes (if applicable)

  • Permissions checked
  • Endpoint tests added

Additional Notes

[Any other relevant information]

… enabled

Previously, the GeoMap component always included radius in emitted values
even when radiusControls was disabled, causing unwanted diffs in System
Settings when only updating map center coordinates.
@cango91 cango91 self-assigned this Jul 4, 2025
@cango91 cango91 requested a review from level09 July 5, 2025 13:38
Copy link
Collaborator

@level09 level09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great

@cango91 cango91 changed the title BYNT-1383: Fix GeoMap component to only emit radius when controls are enabled BYNT-1383: Set default for radius Jul 6, 2025
@tarekio tarekio merged commit 7071bc2 into main Jul 9, 2025
6 of 7 checks passed
@tarekio tarekio deleted the bynt-1383-map-radius-fix branch July 9, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants