Conversation
example/src/screens/HomeScreen.tsx
Outdated
| }); | ||
| }; | ||
|
|
||
| const invalidateCache = () => { |
There was a problem hiding this comment.
Esto moveuse de sitio, pero creo que non pertence ao bloque "share live location"
There was a problem hiding this comment.
Correcto, fixeno a última hora e arrastre máis do que debía
| }); | ||
| }; | ||
|
|
||
| // eslint-disable-next-line @typescript-eslint/no-shadow |
There was a problem hiding this comment.
Por que ocultamos un problema que nos está sinalando o lint?
CHANGELOG_UNRELEASED.md
Outdated
| @@ -0,0 +1,2 @@ | |||
| - Added new method MapViewRef.setShareLiveLocationSession to set the session identifier of the Share Live Location Session the MapView should display. | |||
There was a problem hiding this comment.
Eu dedicaríalle unha boa explicación a como funciona esto: hai dous roles, un escupe posicións, outro consume, tal método é para tal cousa, a outra xa se fai coa UI do viewer, etc.
There was a problem hiding this comment.
Esta tarefa ten que acabar nunha doc da web onde se explique en que consiste, como se habilita a funcionalidade dende web, e que metodo nativo hai que chamar en cada plataforma para poder setear o id de sesión. Despois habería que linkala nos changelogs.
Cando este feita esa doc, haberia que actualizar os changelogs publicos. Está xa indicado na tarefa que é preciso facer esa doc e polo tanto debese validar coa doc
There was a problem hiding this comment.
Joe, que maestria para esquivar balas colega. @fernandosanchez-situm tesme que dar unhas clases jajajajajaja
There was a problem hiding this comment.
jaja, non que hai que facer esa doc. Eso non é esquivar, é correr diante da bala, terminache dando
plugin/src/wayfinding/types/index.ts
Outdated
| navigateToCar: (params?: NavigateToCarPayload) => void; | ||
|
|
||
| /** | ||
| * Establishes the share live location session the mapView should display. |
There was a problem hiding this comment.
O mismo que abaixo. Explicaría ben todo o tinglado porque non é doado de entender e creo que nós mismos nos imos olvidar.
There was a problem hiding this comment.
Hmm, aqui cambia un pouco a cousa, xa que se precisaria un novo commit e release para actualizado.
| interface LocationAPI { | ||
| startPositioning: (locationRequest?: LocationRequest) => void; | ||
| stopPositioning: (callback: (response: { success: boolean }) => void) => void; | ||
| startShareLiveLocation: (options: any) => void; |
There was a problem hiding this comment.
Aquí definiría el tipo que espera
There was a problem hiding this comment.
É un metodo interno, que non queremos que use ningen, en parte esta feito por iso
plugin/src/wayfinding/types/index.ts
Outdated
|
|
||
| /** | ||
| * Establishes the share live location session the mapView should display. | ||
| * A share live location session allows the mapView to display the real-time location of another user. |
There was a problem hiding this comment.
E solo para ensenhar no teu viewer a posicion real-time de outro usuario ? Realmente non seria para crear unha sesion na que 2 usuarios poden verse mutuamente a sua ubicacion ? 2 usuarios solo ou poden ser varios ?
There was a problem hiding this comment.
Non se ven mutuamente, é como o de whassap, ti compartes ubicación con alguen e ese alguen veche a ti pero ti non a el.
A funcionaliade implementada por web de momento é para un só, pero entendo que a futuro poderia chegar un add
5 participants
No description provided.