Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Live location sharing - extract location markers into generic Marker #8225

Merged
merged 5 commits into from
Apr 6, 2022

Conversation

kerryarchibald
Copy link
Contributor

@kerryarchibald kerryarchibald commented Apr 4, 2022

In timeline:

Pin drop:
Screenshot 2022-04-04 at 16 51 32

Static user location:
Screenshot 2022-04-04 at 17 19 18

Live user location (not implemented yet)
Screenshot 2022-04-04 at 16 51 07

Share dialog:

Pin drop:
Screenshot 2022-04-04 at 17 18 43

Static user location:
Screenshot 2022-04-04 at 17 18 17

Live user location:
Screenshot 2022-04-04 at 17 18 26


This change is marked as an internal change (Task), so will not be included in the changelog.

Preview: https://pr8225--matrix-react-sdk.netlify.app
⚠️ Do you trust the author of this PR? Maybe this build will steal your keys or give you malware. Exercise caution. Use test accounts.

Kerry Archibald added 2 commits April 4, 2022 17:17
Signed-off-by: Kerry Archibald <[email protected]>
@kerryarchibald kerryarchibald requested a review from a team as a code owner April 4, 2022 15:23
@kerryarchibald kerryarchibald added the T-Task Refactoring, enabling or disabling functionality, other engineering tasks label Apr 4, 2022
@codecov
Copy link

codecov bot commented Apr 4, 2022

Codecov Report

Merging #8225 (cf85fde) into develop (efb44e1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff            @@
##           develop    #8225   +/-   ##
========================================
  Coverage    28.27%   28.28%           
========================================
  Files          861      862    +1     
  Lines        49800    49801    +1     
  Branches     12688    12689    +1     
========================================
+ Hits         14083    14085    +2     
+ Misses       35717    35716    -1     
Impacted Files Coverage Δ
src/components/views/location/LocationPicker.tsx 91.86% <100.00%> (-0.19%) ⬇️
src/components/views/location/Marker.tsx 100.00% <100.00%> (ø)
src/components/views/messages/MLocationBody.tsx 70.37% <100.00%> (+0.72%) ⬆️

@kerryarchibald kerryarchibald enabled auto-merge (squash) April 6, 2022 11:33
@kerryarchibald kerryarchibald merged commit b987390 into develop Apr 6, 2022
@kerryarchibald kerryarchibald deleted the psf-880/generic-location-marker branch April 6, 2022 11:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
T-Task Refactoring, enabling or disabling functionality, other engineering tasks
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants