Skip to content

fix: leaflet-draw rectangle bug#1144

Merged
mikkojamG merged 2 commits intomainfrom
fix/hearing-map-rectangle-bug
Apr 15, 2025
Merged

fix: leaflet-draw rectangle bug#1144
mikkojamG merged 2 commits intomainfrom
fix/hearing-map-rectangle-bug

Conversation

@mikkojamG
Copy link
Contributor

@mikkojamG mikkojamG commented Apr 14, 2025

Fixing react-leaflet-draw rectangle bug.

@mikkojamG mikkojamG force-pushed the fix/hearing-map-rectangle-bug branch from 38db385 to aa5a67f Compare April 14, 2025 10:35
@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

@mikkojamG mikkojamG requested review from a team, antervon-gofore and arpargo April 14, 2025 10:47
@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

e2e tests result is failed for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

arpargo
arpargo suggested changes Apr 14, 2025
View reviewed changes
Copy link
Contributor

@arpargo arpargo left a comment

Choose a reason for hiding this comment

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

Muuten vaikuttaa toimivalta. Yksi kommentti liittyen CommentFormMappiin

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

e2e tests result is success for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😆🎉🎉🎉

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

e2e tests result is failed for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

e2e tests result is success for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😆🎉🎉🎉

@mikkojamG mikkojamG force-pushed the fix/hearing-map-rectangle-bug branch from 0b057be to 58a3c12 Compare April 14, 2025 12:11
@mikkojamG mikkojamG requested a review from arpargo April 14, 2025 12:20
@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

e2e tests result is failed for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😿💢💥💥

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 14, 2025

e2e tests result is success for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😆🎉🎉🎉

@mikkojamG mikkojamG force-pushed the fix/hearing-map-rectangle-bug branch from 58a3c12 to 6f748a0 Compare April 15, 2025 04:52
@sonarqubecloud
Copy link

sonarqubecloud bot commented Apr 15, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
66.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 15, 2025

KERROKANTASI-UI branch is deployed to platta: https://kerrokantasi-pr1144-ui.dev.hel.ninja 🚀🚀🚀

@terovirtanen
Copy link
Contributor

terovirtanen commented Apr 15, 2025

e2e tests result is success for https://kerrokantasi-pr1144-ui.dev.hel.ninja 😆🎉🎉🎉

@mikkojamG mikkojamG merged commit 4975a01 into main Apr 15, 2025
23 checks passed
@mikkojamG mikkojamG deleted the fix/hearing-map-rectangle-bug branch April 15, 2025 08:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@antervon-gofore antervon-gofore Awaiting requested review from antervon-gofore

1 more reviewer

@arpargo arpargo arpargo approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mikkojamG @terovirtanen @arpargo