Plots2: possibilité d'actualiser les données affichées des cartes en ligne lorsque vous effectuez un panoramique/zoom

Créé le 8 janv. 2018  ·  20Commentaires  ·  Source: publiclab/plots2

Veuillez décrire le problème (ou l'idée)

Vous pouvez intégrer des cartes en ligne à l'aide de ce powertag en ligne : https://publiclab.org/wiki/inline-maps

Le code qui pilote ceci est ici :

https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124

Et le modèle est ici : https://github.com/jywarren/plots2/blob/master/app/views/map/_leaflet.html.erb

Cependant, lorsque vous faites glisser la carte, les marqueurs ne sont pas actualisés pour la nouvelle zone d'affichage.

Brisons cela en trois :

  1. [x] faisons un appel API pour les données près d'un point, similaire à https://publiclab.org/api/srch/notes?srchString=foo mais peut-être https://publiclab.org/api/srch/notes?geo =41,0 ,-91,3. Cela pourrait être construit à partir de https://github.com/publiclab/plots2/blob/master/app/api/srch/search.rb#L52 -L72, ou une nouvelle méthode pourrait être créée
  2. [x] recâchons https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124 pour charger une carte vierge qui charge ensuite l'appel API pour le point central, et affiche des notes sous forme de points sur les nouvelles de l'API
  3. [x] définissons la carte en ligne pour qu'elle récupère les notes près de son point central chaque fois que la carte est déplacée ou zoomée

Il s'agit d'un gros problème en plusieurs parties que nous pouvons résoudre en trois étapes ou plus. Quelqu'un est-il intéressé à sortir le premier?

JavaScript Ruby break-me-up enhancement help wanted

Commentaire le plus utile

plugin génial -- oui parfait !

Le vendredi 30 mars 2018 à 16h07, Sagarpreet Chadha < [email protected]

a écrit:

@jywarren https://github.com/jywarren , @Gauravano
https://github.com/Gauravano ! Puis-je également résoudre ce problème ?
Le code pourra être utilisé plus tard si nous décidons de créer une bibliothèque.
Qu'est-ce que tu penses ?

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-377611018 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ9bk4gUyLKBbJxS-Qj76NBQY8i2bks5tjpCUgaJpZM4RXAwK
.

Tous les 20 commentaires

@jywarren je veux essayer celui-ci

salut, c'est un gros projet mais je serais heureux de t'aider à l'aborder petit à petit
pièce. Merci et ouvrez un PR le plus tôt possible ! Merci!!

Le mardi 9 janvier 2018 à 04h38, Gaurav Sachdeva [email protected]
a écrit:

@jywarren https://github.com/jywarren Je veux essayer celui-ci

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-356231179 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJznVKoASY6PwMeUjiG4l7JCpM8vwks5tIzOmgaJpZM4RXAwK
.

Salut @jywarren , j'étais en train de considérer toutes les étapes comme indiqué par vous. Je pense (pas sûr) que nous pouvons changer le flux comme - au lieu d'appeler l'API pour les points proches, ne pouvons-nous pas modifier https://github. com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124 tel qu'il recevra le point central de notre part et nous fournira des points proches et chaque fois que le panoramique/zoom est détecté, nous appellerons la fonction .La balise inline sera traitée en créant une fonction supplémentaire qui ne doit être utilisée que lorsque nous devons charger la page et à d'autres moments, nous donnerons simplement les coordonnées du centre pour fonctionner.

Que penses-tu de cela?

Hmm, eh bien, vous avez définitivement mis en évidence la section de code que nous voulons. Je pense que nous
pourrait faire une autre fonction pour cela (c'est peut-être ce que vous dites)
comme "Node.near()" ou quelque chose, appelez-le ici, mais fournissez également un moyen
le côté client peut l'appeler aussi. Mais cette dernière façon de l'appeler extérieurement
semble que cela devrait être via l'API, une interface plus universelle que nous pouvons
offrir à d'autres pour utilisation. Qu'en penses-tu ?

Le 10 janvier 2018 17:38, "Gaurav Sachdeva" [email protected] a écrit :

Salut @jywarren https://github.com/jywarren , je pensais juste à tout
les étapes comme indiqué par vous.Je pense (pas sûr) que nous pouvons changer le flux comme -
au lieu d'appeler l'API pour les points proches, ne pouvons-nous pas modifier
https://github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df
2838528522/app/models/concerns/node_shared.rb#L102-L124 tel qu'il sera
recevoir le point central de notre part et nous fournir des points proches et à chaque fois
pan/zoom est détecté, nous appellerions la fonction. La balise en ligne sera
traité en créant une fonction supplémentaire qui ne doit être utilisée que lorsque nous
devons charger la page et à d'autres moments, nous donnerons simplement le centre
coordonnées pour fonctionner.

Que penses-tu de cela?

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-356761050 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ7Dnhw745kz9E6P67nx_q16MihNpks5tJTu0gaJpZM4RXAwK
.

Hmm... créer une API peut certainement être utile pour les implications futures de cette fonctionnalité, donc d'accord, je m'en tiendrai à votre flux indiqué ci-dessus et ouvrirai bientôt le pull pour la première partie. En outre, veuillez suggérer des modifications dans # 1935. Merci.

Salut Jeff, j'ai utilisé https://github.com/publiclab/plots2/blob/01fac7cda6ab3df0ee9ba492f489edeef741ece9/app/models/concerns/node_shared.rb#L122 -L146 pour récupérer les coordonnées, la fonction donnée sur le lien de votre dépôt :/ /github.com/jywarren/plots2/blob/184eced60d9d6842cad8df469bb1df2838528522/app/models/concerns/node_shared.rb#L102 -L124 utilise le modèle DrupalNodeCommunityTag qui n'est pas utilisé actuellement

Ah, c'est vrai, merci ! Nous l'avons mis à jour depuis.

@jywarren et @Gauravano , je me demandais si je

Je pense que tu pourrais commencer à travailler le 2 pendant que le premier se termine,
avec un RP ; peut-être que @Gauravano aimerait prendre la dernière partie ?

Merci! Il y aura aussi une 4ème partie sur un champ de recherche pour rechercher la carte
emplacements supplémentaires sur lesquels zoomer.

Le 13 janvier 2018 15:18, "Sagarpreet Chadha" [email protected]
a écrit:

@jywarren https://github.com/jywarren et @Gauravano
https://github.com/gauravano , je me demandais si je pouvais faire le 2ème et
3ème partie de ce numéro ? Ce sera passionnant !! 😄

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-357464375 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ1UXCiij_f-iYkgYLuypqD_FHr19ks5tKQ9egaJpZM4RXAwK
.

D'accord, je travaille sur le deuxième. Merci .

Salut @jywarren et @Gauravano , le format JSON de la nouvelle API est :

screen shot 2018-01-16 at 6 46 45 pm

Pour ajouter des marqueurs aux cartes, nous aurons également besoin des valeurs Latitude et Longitude de chaque nœud.
@Gauravano Pouvez-vous également ajouter la valeur de latitude et de longitude de chaque nœud également dans l'API ? Merci !

En fait @jywarren, je viens de noter que l'API que j'ai construite récupère tous les nœuds, mais que se passe-t-il si quelqu'un sur la page wiki utilisait une balise en ligne comme [map: tag:infragram : lat:lon ] alors, nous n'avons pas d'API pour ça. Donc, nous devons avoir 2 URL différentes, une pour les notes et une pour les notes avec étiquette. Qu'en pensez-vous ? Est-ce que je parle bien ?

Bon appel -- je pense que vous avez raison, mais nous pouvons commencer la mise en œuvre sur le premier type de requête, plus simple, voir comment il fonctionne, puis créer une tâche de suivi pour celles basées sur des balises.

(Si vous avez besoin de diviser une nouvelle vue pour la carte actualisable, c'est bien - il est probablement bon d'avoir une implémentation plus simple présente tant qu'elle fonctionne de manière très similaire !)

Modularité ! :-)

Ok alors @sagarpreet-chadha selon vos besoins, je mets à jour l'API pour l'instant

OK, l'étape API est terminée et fusionnée ! Réouverture pour les parties restantes :-)

Nous pouvons utiliser ce plugin : http://labs.easyblog.it/maps/leaflet-search/ pour rechercher des emplacements sur des cartes (d'après votre commentaire ici ). Qu'en penses-tu @jywarren ?

@jywarren , @Gauravano ! Puis-je également résoudre ce problème ?
Le code pourra être utilisé plus tard si nous décidons de créer une bibliothèque.
Qu'est-ce que tu penses ?

plugin génial -- oui parfait !

Le vendredi 30 mars 2018 à 16h07, Sagarpreet Chadha < [email protected]

a écrit:

@jywarren https://github.com/jywarren , @Gauravano
https://github.com/Gauravano ! Puis-je également résoudre ce problème ?
Le code pourra être utilisé plus tard si nous décidons de créer une bibliothèque.
Qu'est-ce que tu penses ?

-
Vous recevez ceci parce que vous avez été mentionné.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/1934#issuecomment-377611018 ,
ou couper le fil
https://github.com/notifications/unsubscribe-auth/AABfJ9bk4gUyLKBbJxS-Qj76NBQY8i2bks5tjpCUgaJpZM4RXAwK
.

Ce serait vraiment génial de reprendre !

https://publiclab.org/wiki/inline-maps

Résolu en #4382

Cette page vous a été utile?
0 / 5 - 0 notes