Skip to content

RES-1863 Add draggable map for group create#637

Closed
edwh wants to merge 18 commits into
developfrom
RES-1862_draggable_map
Closed

RES-1863 Add draggable map for group create#637
edwh wants to merge 18 commits into
developfrom
RES-1862_draggable_map

Conversation

@edwh

@edwh edwh commented Apr 11, 2023

Copy link
Copy Markdown
Collaborator
  1. Add a draggable map component. This will be shared later with events (which isn't in Vue yet).
  2. Use this component from group create/edit.
  3. Allow the API call to take a lat/lng from the client if passed; otherwise we fall back to the geocoded location.

@edwh
edwh requested a review from restart-neil April 11, 2023 10:42
edwh and others added 3 commits April 19, 2023 09:15
@edwh edwh changed the title RES-1862 Add draggable map for group create RES-1863 Add draggable map for group create May 17, 2023
edwh added 3 commits May 17, 2023 11:48
@sonarqubecloud

Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2023

Copy link
Copy Markdown

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot E 2 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.6% 0.6% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@edwh

edwh commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator Author

Superseded by #895, which re-implements this on the current codebase (the static location map and Places autocomplete having landed in the meantime, only the draggable fine-tuning and the API lat/lng override remained). The Mapbox geocoder switch that was bundled in here belongs to RES-1858 / #644.

@edwh edwh closed this Jul 14, 2026
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.

2 participants