Plots2: Planification de projets de sensibilisation

Créé le 2 déc. 2019  ·  154Commentaires  ·  Source: publiclab/plots2

Tout d'abord, encore félicitations et merci d'avoir choisi Public Lab pour votre bourse Outreachy ! :tada: :tada:

Je voulais commencer à aligner nos plans de projets respectifs ; d'abord parce que nous aurons deux boursiers qui travailleront ensemble sur ce sujet, et deuxièmement, parce qu'il y a eu et qu'il y aura encore d'autres projets parallèles et qui se chevauchent avec des liens avec votre travail.

Ce projet s'inscrit dans un ensemble plus large de « systèmes géographiques » (https://github.com/publiclab/plots2/milestone/7) qui ont également été travaillés par @ananyaarun (lors du dernier cycle d'Outreachy) et d'autres également, y compris @sagarpreet-chadha, @cesswairimu , et plus, à la fois ici dans plots2 et dans Leaflet Environmental Layers !

@crisner a fait du bon travail sur la bibliothèque LEL, tandis que @nstjean a fait du bon travail sur le côté plots2 des choses, donc cela semble être, potentiellement, un arrangement complémentaire naturel. Alors, discutons un peu de chaque côté et de l'emplacement des chevauchements !

Propositions originales

Ressources

Quelques ressources de départ - c'est une bonne idée de lire notre guide de style et notre feuille de

OK, je prends trop de temps pour poster ceci, donc je vais juste cliquer sur Publier et ajouter plus dans un instant.

Bienvenue à @crisner @nstjean et aux mentors Outreachy ! @ananyaarun @rexagod @cesswairimu @aSquare14 @IshaGupta18 @MayankKashyap !

👋 🙌 🎉 💯 🔥

discussion outreachy planning

Commentaire le plus utile

Je suis tellement excitée qu'hier j'ai eu du succès avec les tests Jasmine ! Ce furent de longues journées d'essais et d'échecs répétés.
https://github.com/publiclab/leaflet-blurred-location/pull/235 est le correctif de LBL
https://github.com/publiclab/leaflet-blurred-location-display/pull/93 est le correctif de LBLD - peut ne pas être nécessaire
Et aujourd'hui, je vais examiner les échecs des tests de LEL et de la couche People. Je parie que c'est lié.

Tous les 154 commentaires

J'ai organisé un problème de planification de type liste ici, avec de nombreux problèmes liés liés et regroupés ; Je vais essayer d'organiser ça aujourd'hui ! https://github.com/publiclab/plots2/issues/6801

Je voulais également signaler cette excellente ressource sur le test de votre travail sur le serveur unstable - c'est un excellent moyen de voir comment les choses fonctionnent dans un environnement presque identique au serveur de production ! https://github.com/publiclab/plots2/blob/master/doc/TESTING.md#testing -your-work-with-unstable

Il y a vraiment beaucoup de pièces mobiles différentes dans ce projet, alors posez des questions si vous êtes perdu ou confus ! Ou voir un moyen de faire des choses qui seraient mieux !

Par exemple, voici un exemple d'utilisation de LEL dans plots2 qui ne fonctionne pas correctement : https://github.com/publiclab/plots2/issues/6889

D'accord, super!

Une question que j'ai: dans ma proposition, j'ai un plan pour une nouvelle façon de définir l'emplacement sur les profils des utilisateurs. Voulez-vous toujours que je le fasse, même si ce n'est pas sur les listes de contrôle de plots2 ?

Sûr! Je pense que nous devrons tous mettre à jour/affiner/réviser les plans de projet et j'adorerais créer une nouvelle idée de conception en atelier dans le cadre de celui-ci ! De plus, les listes que j'ai ajoutées jusqu'à présent ne sont en aucun cas complètes ou exhaustives.

Je voulais partager ce problème "comment nous travaillons" qui est un peu dense mais pourrait avoir des informations utiles pour vous deux : https://github.com/publiclab/plots2/issues/5667

OK, donc @nstjean a identifié de nombreuses corrections de bugs et améliorations de l'interface utilisateur pour la saisie de l'emplacement dans la phase 1 - elles ont toutes l'air SUPER. Assurez-vous simplement que le style correspond à notre guide de style (en particulier le style "Ajouter un emplacement" que nous avons essayé de standardiser - un bouton blanc avec un marqueur rouge - #6856) et le tableau de bord est sur le point d'être reconstruit #6787 mais y conserve également un espace pour le bouton Ajouter votre emplacement. Commençons par lister ces idées/tâches dans notre article de liste de contrôle sur https://github.com/publiclab/plots2/issues/6801 et nous pouvons les marquer comme terminées au fur et à mesure !

@crisner a prévu un travail impressionnant pour le sélecteur/contrôle/menu de couche. Je me demande si @crisner peut vouloir prioriser le débogage du search-for-location dans le sélecteur d'emplacement (qui utilise cette bibliothèque : https://github.com/publiclab/leaflet-blurred-location) et les calques "navigateur" menu, basé sur le HTML statique dans https://github.com/publiclab/leaflet-environmental-layers/issues/232. Une fonctionnalité qui nous intéresse vraiment est une démo dans la démo Github Pages qui peut générer un code d'intégration - essentiellement lorsque vous effectuez un panoramique et un zoom et sélectionnez et désélectionnez des calques, cela modifierait le hachage d'URL, et vous seriez en mesure d'obtenir l'URL de cela. Les deux utilisant une méthode comme getUrlHash() dans la bibliothèque, ainsi que dans l'interface utilisateur. Ce qui est bien, c'est que @sagarpreet-chadha a déjà implémenté cela dans le contexte de plots2 , de sorte que le travail pourrait être rétroporté sur LEL, peut-être ? https://github.com/publiclab/plots2/pull/6874

La raison pour laquelle je suggère cela est que le sélecteur de calques est déjà au début de votre timeline @crisner et cela

Je vous recommande également de parcourir les propositions des autres et de réfléchir à la manière dont, au fur et à mesure que vous vous lancez dans le projet, vous pourriez répartir le travail pour les sections intermédiaires qui se chevauchent le plus ; par exemple, vous avez tous les deux partagé de très bonnes maquettes d'interface utilisateur avec des idées sur l'endroit où les couches actives peuvent être affichées et d'autres éléments. Si nous faisons du bon travail en développant des choses en petites pièces modulaires , nous devrions être capables de travailler en synchronie sur ces choses.

Enfin, un domaine dans lequel nous pourrions nous développer, étant donné que nous avons actuellement 2 personnes travaillant sur ce projet, est celui des tests - à la fois dans plots2 et dans les tests d'interface utilisateur dans LEL, dont certains sont ici : https : //github.com/publiclab/leaflet-environmental-layers/tree/master/spec/javascripts

Ces caractéristiques sont complexes et profondément imbriquées. Nous pouvons gagner beaucoup en stabilité si nous protégeons les fonctionnalités (et les corrections de bugs) avec de bons tests, alors j'aimerais savoir comment vous pouvez tous les deux prévoir du temps pour intégrer les tests dans vos plans de projet ! Voici un problème où nous pouvons également nous organiser à ce sujet : https://github.com/publiclab/leaflet-environmental-layers/issues/71

Merci beaucoup, je sais que je viens de partager une tonne. Prenez votre temps pour traiter ces informations, et n'hésitez pas à poser des questions, ou à indiquer où j'ai manqué des choses ou où je n'ai pas été clair, ou où vous avez de nouvelles et bonnes idées à partager !

Merci beaucoup d'avoir travaillé avec nous !!!!!

Oui ça sonne bien !! Je vais prendre le temps de parcourir toutes les différentes listes et propositions et @crisner et moi déciderons comment coordonner nos efforts !

Merci d'avoir présenté toutes les informations @jywarren ! Je vais commencer par passer en revue toutes les informations et nos propositions de projets. Je m'assurerai de demander si quelque chose n'est pas clair. Je vais avoir une discussion avec @nstjean et planifier notre travail en conséquence.

C'est super..heureux de travailler avec vous deux..n'hésitez pas à me taguer en cas de quoi que ce soit. Merci

Je viens également d'ajouter @crisner @nstjean à l'équipe @publiclab/soc (que nous utilisons pour divers programmes de style Summer of Code) ce qui devrait vous donner considérablement plus d'accès ; pour étiqueter, éditer et trier les problèmes et les demandes d'extraction ! Bienvenue! ??

J'ai répertorié quelques tâches ci-dessous pour l'instant et j'ai essayé de les lier aux problèmes existants. J'espère finir de les disposer d'ici demain en fonction des tâches de nos propositions. @nstjean , veuillez modifier ceci pour ajouter les tâches que vous avez définies. @jywarren J'ai quelques questions que j'ai incluses à la fin.

[Natalie] Nous avons ajouté tous les éléments de notre plan plots2.

Parcelles2

Toutes les tâches pour /map/route sont ici : #6993

  • [x] Les permaliens de la couche de carte sont écrasés à l'ouverture de la page (problème #6889) PR #293

  • [x] Ajoutez une balise de puissance zoom:xx pour le zoom de la carte ainsi que la latitude et la longitude lors de l'enregistrement de l'emplacement #7019 PR #7020

  • [x] Activer les cartes intégrées dans les pages wiki, les notes et les questions

  • [x] Ajouter une invite au tableau de bord (/dashboard) pour ajouter l'emplacement de l'utilisateur, si l'utilisateur n'a pas défini d'emplacement.

  • [x] Correction de la saisie de l'emplacement sur /people/ #5882 PR https://github.com/publiclab/plots2/pull/6895

  • [x] Correction du marquage de l'emplacement dans les profils utilisateur PR #6886

Mettre à jour le formulaire de localisation « nouveau message » :

  • [x] Correction d'une erreur de gel de clic et de panoramique
  • [x] Déboguer la recherche de localisation dans le sélecteur de localisation https://github.com/publiclab/leaflet-blurred-location/issues/214 et #6915
  • [x] Mettre à jour le style en fonction de la proposition de Natalie #6968 PR #6969
  • [x] Ajouter l'emplacement par défaut de l'emplacement du profil de l'utilisateur, permettre à l'utilisateur de le supprimer ou de le modifier #6946 PR #6957
  • [x] Ajouter une vérification de la balise de zoom, si elle n'est pas présente, calculer le niveau de zoom à partir de lat et de la longueur et enregistrer le zoom - inclus dans https://github.com/publiclab/plots2/pull/6957

Projet : Refonte du modal popup de localisation

La proposition de Natalie incluait une conception possible pour l'entrée de l'emplacement du profil de l'utilisateur, cela devra être publié et des commentaires recueillis quant à la meilleure façon de l'accomplir, comment il devrait fonctionner et ressembler.

Essai:

  • [x] Correction de l'échec des nouveaux tests système en raison de problèmes de synchronisation #6904 PR #6951

PL.Editeur

LBL

LIE

  • [x] Travail sur la fonctionnalité d'intégration PR #296

  • [x] Faire du nom de domaine une option paramétrable par défaut sur gh-pages PR # 307

  • [x] Afficher les directions en haut de la carte dans la page de démonstration PR #309

  • [x] Remplacez le contrôle des calques de feuillets par le menu du navigateur de calques à partir de la maquette dans #232 PR #320

  • [x] Collecter les URL des « requêtes » à partir des couches si elles sont ouvertes aux soumissions de la communauté #138 PR #317

    • [x] Implémentez la fonctionnalité de rapport dans le menu du navigateur de couches pour les couches qui acceptent des rapports comme OdorLog PR #320
  • [x] Cloner la page de démonstration sur embed.html afin que les intégrations réelles n'affichent pas l'avis PR #311

  • [x] Suivre le cadre de délimitation de chaque calque et afficher uniquement ceux qui se croisent avec la vue actuelle #117

  • [x] Afficher la marque "nouveaux éléments" dans le menu Calques lorsque vous entrez dans une zone avec un nouveau calque #133

  • [x] Implémentez le mode minimal comme indiqué dans le numéro 123 .

  • [x] Affiche un bouton pour basculer entre le mode par défaut et le mode minimal. :ampoule:

    • [x] Implémenter une fonction bascule
  • [ ] Afficher les légendes facultatives pour chaque couche #103

  • [ ] Arrêtez d'afficher les spinners si un calque ne se charge pas pendant 10 secondes (problème #290 )

  • [x] La recherche n'est pas ouverte par défaut mais derrière une icône en forme de loupe qui s'ouvre PR #326

  • [x] Implémenter une fonctionnalité de recherche pour rechercher un emplacement sur lequel se recentrer avec un bouton de recherche et un géocodeur de saisie semi-automatique (discuté dans #134 ) PR #326

  • [ ] Zoomez sur la carte en fonction de l'emplacement de l'utilisateur à l'aide de la géolocalisation.

  • [ ] Implémenter une fonctionnalité d'aide :bulb:

    • [ ] :bulb : option d'affichage permettant aux utilisateurs de déposer des rapports de bogue avec le bouton dirigeant les utilisateurs vers un modèle de problème de bogue

    • [ ] Option pour demander une nouvelle couche qui dirige vers un modèle de problème. -> Processus de soumission de couche, métadonnées par couche #135

    • [ ] :bulb : une option pour afficher des informations sur les différents éléments de l'interface utilisateur de la carte (peut être utile aux nouveaux utilisateurs) peut être sous forme de modale ou de simple faq ou sous forme d'info-bulles pas à pas qui peuvent être ignorées.

  • [ ] Implémentez un menu de fonctionnalités qui peut basculer individuellement d'autres éléments de l'interface de la carte (à l'exception du menu des couches) à partir de la vue. comme une bonne idée à l'époque) :bulb:

  • [ ] Créer une fonction pour faciliter l'ajout de calques

    • [ ] Réduisez le code de couche redondant. Un membre travaille sur un plugin qui permet d'éviter le code répétitif lors de l'ajout de nouvelles couches qui est mentionné dans #276 . On pourrait essayer d'utiliser ça.
  • [ ] Méthodes standard pour chaque couche comme layer.show() layer.hide() layer.isVisible() #251

  • [ ] Créer une fonction d'agrégation de noms de calques #253

  • [ ] Créez une option pour basculer la fonction spiderfier des marqueurs qui se chevauchent pour les calques individuels :bulb:

  • [x] Faire en sorte que la liste des noms de calques dans README et les chaînes de hachage des calques soient cohérentes PR # 315

  • [x] Correction de la sélection de couche de base active PR #300

  • [x] Reformatez/nettoyez le code PR #303

Essai

  • [x] Ecrire des tests pour le contrôle embarqué PR #314

  • [ ] Écrire des tests d'interface utilisateur

Des questions

  • À quoi sert « partagez vos propres données cartographiques » dans le menu du navigateur de couches ? Montré ici dans la maquette du numéro 232 .

  • Il y avait une mention sur le débogage de la recherche d'emplacement dans le sélecteur d'emplacement. Est-ce pour la section "Ajouter un emplacement" lors de l'ajout d'un message ?

  • Comment le menu « navigateur » des couches doit-il être implémenté dans LEL ? Mon idée était de l'activer en cliquant sur l'icône d'alerte affichant les calques disponibles.

  • J'avais proposé quelques nouvelles fonctionnalités pour l'interface utilisateur LEL dans ma proposition de projet. Est-ce que je les inclus aussi en tant que tâches ici ?

  • Je pense travailler sur la fonctionnalité d'intégration et le menu des calques pour la première semaine, puis écrire des tests pour eux. Y a-t-il des tâches que je devrais envisager de donner la priorité aux autres ?

Merci. :-)

J'ai mis à jour le post de

Nous avons eu un certain chevauchement dans LEL et nous devrons déterminer qui prendra quoi ! De plus, dans le problème de planification des entités géographiques pour les parcelles 2, il y a plus de tâches pour les parcelles 2, et dans le problème de planification pour LEL, il y en a également plus. J'ai créé une grande liste principale de tous les problèmes à l'aide d'un outil de planification des tâches pour essayer de les visualiser tous. J'ai envoyé l'info à @crisner pour qu'elle puisse l'utiliser aussi. :)

Une question que j'ai concerne les outils de localisation dans Plots2. Je remarque qu'à certains endroits, il utilise le modal contextuel, comme dans votre profil lorsque vous ajoutez un emplacement. Mais ensuite, lorsque vous écrivez un nouveau message ou une nouvelle page, un formulaire en ligne s'affiche.

  1. Y a-t-il une raison d'avoir deux formulaires de saisie de localisation différents ?
  2. Doit-on se regrouper dans l'un ou l'autre ?

Salut tout le monde! Excellentes questions, et super d'avoir de vos nouvelles ! Je vais juste sauter dedans :

À quoi sert « partagez vos propres données cartographiques » dans le menu du navigateur de couches ? Montré ici dans la maquette du numéro 232.

Idéalement, cela mène à une page qui présente la LEL et présente des étapes simples pour suggérer/soumettre une nouvelle idée de couche (url, type, etc.) afin que nous ajoutions progressivement plus de couches. Nous n'avons pas encore de moyen simple pour les non-développeurs de suggérer des couches, mais nous avons une étiquette new-layer ... en attendant, cela pourrait conduire à une section du README que nous développerons plus tard.

Il y avait une mention sur le débogage de la recherche d'emplacement dans le sélecteur d'emplacement. Est-ce pour la section "Ajouter un emplacement" lors de l'ajout d'un message ?

Je pense que oui! J'ai vu quelques choses - l'une, qu'il se trompe près de la limite 360 ​​/ 0 ° https://github.com/publiclab/leaflet-blurred-location/issues/213 et une à peu près que le modal est vraiment lent à apparaître https ://github.com/publiclab/plots2/issues/6897

Comment le menu « navigateur » des couches doit-il être implémenté dans LEL ? Mon idée était de l'activer en cliquant sur l'icône d'alerte affichant les calques disponibles.

Ouais, je ne suis pas sûr à 100%, et ouvert aux idées. Je pense que le contrôle de sélection de calque actuel en haut à droite contient peu d'informations pour ne pas être vraiment très utile. Il n'y a pas d'informations supplémentaires sur chaque couche, elles sont nommées de manière obscure, etc. Ainsi, peut-être que le plus grand menu le remplacerait entièrement et s'activerait lorsque vous cliquez sur ce bouton dans le coin ?

J'avais proposé quelques nouvelles fonctionnalités pour l'interface utilisateur LEL dans ma proposition de projet. Est-ce que je les inclus aussi en tant que tâches ici ?

Ce serait formidable, et pouvez-vous également les mettre en évidence spécifiquement pour les commentaires ?

Je pense travailler sur la fonctionnalité d'intégration et le menu des calques pour la première semaine, puis écrire des tests pour eux. Y a-t-il des tâches que je devrais envisager de donner la priorité aux autres ?

Cela sonne plutôt bien ! Notez également la relation entre la génération d'un code d'intégration et la génération d'un code court plots2, alias un "inline powertag" (que nous utilisons pour intégrer des cartes dans plots2, et est expliqué ici : https://publiclab.org/wiki/inline-maps ). Celles-ci sont très similaires voire redondantes, bien que nous n'utilisions pas d'iframes sur les plots2, ce qui est bien. Ainsi, la fonction qui génère le code d'intégration pourrait facilement être remplacée par une fonction qui génère un code court pour les tracés2, vous savez ?

Et à @nstjean --

Y a-t-il une raison d'avoir deux formulaires de saisie de localisation différents ?
Doit-on se regrouper dans l'un ou l'autre ?

Ils devraient partager beaucoup et se ressembler, c'est sûr ! Mais l'un est pour l'activité discrète d'ajout d'un emplacement, tandis que l'autre nous voulions apparaître plus clairement comme une étape dans le flux de création d'un article, nous ne voulions donc pas qu'il soit dans un modal contextuel. Est-ce que ça a du sens? Et celui de l'éditeur a également des fonctionnalités supplémentaires telles que la possibilité de récupérer le zoom par défaut et les coordonnées lat/lon à partir des paramètres d'URL, un peu comme la façon dont @nfarve fonctionne ici : https://github.com/publiclab/plots2 /tirer/6858

J'espère que ceux-ci aideront à répondre à vos questions ! S'il vous plaît continuez à demander! Super journée 1 🎉

Et à @nstjean --

Y a-t-il une raison d'avoir deux formulaires de saisie de localisation différents ?
Doit-on se regrouper dans l'un ou l'autre ?

Ils devraient partager beaucoup et se ressembler, c'est sûr ! Mais l'un est pour l'activité discrète d'ajout d'un emplacement, tandis que l'autre nous voulions apparaître plus clairement comme une étape dans le flux de création d'un article, nous ne voulions donc pas qu'il soit dans un modal contextuel. Est-ce que ça a du sens? Et celui de l'éditeur a également des fonctionnalités supplémentaires telles que la possibilité de récupérer le zoom par défaut et les coordonnées lat/lon à partir des paramètres d'URL, un peu comme la façon dont @nfarve fonctionne ici : #6858

Ça a du sens! Je pense qu'ils doivent se ressembler davantage, j'aimerais styliser la version modale contextuelle pour qu'elle ressemble davantage à la version en ligne.

bien sûr, vous voulez partager une maquette dans un problème dans
https://github.com/publiclab/leaflet-blurred-location/ ?

Le mar. 3 déc. 2019 à 15h36 Natalie St Jean [email protected]
a écrit:

Et à @nstjean https://github.com/nstjean --

Y a-t-il une raison d'avoir deux formulaires de saisie de localisation différents ?
Doit-on se regrouper dans l'un ou l'autre ?

Ils devraient partager beaucoup et se ressembler, c'est sûr ! Mais l'un est pour le
activité discrète d'ajout d'un emplacement, tandis que l'autre nous voulions apparaître
plus clairement comme une étape dans le processus de création d'un article, nous ne voulions donc pas
il doit être dans un modal contextuel. Est-ce que ça a du sens? Et celui dans l'éditeur
a également des fonctionnalités supplémentaires telles que la possibilité de récupérer le zoom par défaut et
coordonnées lat/lon à partir des paramètres d'URL, un peu comme @nfarve
https://github.com/nfarve travaille ici : #6858
https://github.com/publiclab/plots2/pull/6858

Ça a du sens! Je pense qu'ils doivent se ressembler davantage, j'aimerais beaucoup
stylisez la version modale contextuelle pour qu'elle ressemble davantage à la version en ligne.

-
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/6887?email_source=notifications&email_token=AAAF6J5MKFQ5EZBJDJNOPVTQW27KHA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW#com2LNMVXW134ZLJK36 ,
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6J3IATZZZW4YPYI234DQW27KHANCNFSM4JTZOL5A
.

Ok, je vais travailler sur une maquette.

Une autre question pendant que j'y réfléchis.

Nos propositions originales avaient un calendrier avec des tâches prévues pour chaque semaine. C'est bon pour la responsabilité et les mentors qui s'assurent que nous sommes sur la bonne voie. Avec deux d'entre nous travaillant sur le même grand projet, préféreriez-vous que nous attribuions les tâches maintenant pendant que nous planifions, afin que nous sachions qui est en charge de quoi ? Ou préféreriez-vous que nous travaillions ensemble sur la liste, en effectuant la tâche suivante sur la liste ?

L'un peut écrire le code et l'autre peut tester ce code. Cela permettra
de meilleurs tests et un meilleur code. Comme Crisner a écrit la fonction A et nstjean
a écrit des tests pour A. Ensuite, crisner a écrit des tests pour la fonction B écrits par
nstjean. Cela encouragera également la programmation en binôme.
Juste une suggestion. Merci

Le mer. 4 déc 2019, 8h36 Natalie St Jean, [email protected]
a écrit:

Une autre question pendant que j'y réfléchis.

Nos propositions originales avaient un calendrier avec des tâches prévues pour chaque semaine.
C'est bon pour la responsabilité et les mentors qui s'assurent que nous sommes sur la bonne voie.
Avec deux d'entre nous travaillant sur le même grand projet, préféreriez-vous que nous attribuions
les tâches maintenant pendant que nous planifions, afin que nous sachions qui est en charge de quoi ? Ou serait
vous nous préférez si nous avons travaillé ensemble sur la liste, quelle que soit la tâche
est le prochain sur la liste ?

-
Vous recevez ceci parce que vous faites partie d'une équipe qui a été mentionnée.
Répondez directement à cet e-mail, consultez-le sur GitHub
https://github.com/publiclab/plots2/issues/6887?email_source=notifications&email_token=AFAAEQY62FEVTDVXZFGWUHTQW4NDJA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJZKTDN5WWAI45
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AFAAEQZ4HQEVGK3WQAMFAT3QW4NDJANCNFSM4JTZOL5A
.

@nstjean J'ai mis à jour la liste.

@jywarren J'ai mis à jour la liste des tâches. Ceux avec l'emoji :bulb: sont les nouvelles fonctionnalités que j'avais mentionnées. J'aimerais avoir votre avis sur leur mise en œuvre.

J'aimerais également avoir vos commentaires sur la tâche ci-dessous.

Zoomez sur la carte en fonction de l'emplacement de l'utilisateur à l'aide de la géolocalisation.

Une autre question. Certaines tâches de la liste n'ont pas leurs propres problèmes. Leur créons-nous un problème avant de commencer à travailler sur ces tâches ?

@nstjean et moi avons organisé les tâches ensemble. Nous allons répartir les tâches au fur et à mesure.

Aurons-nous besoin de publier un calendrier hebdomadaire des tâches, comme dans nos propositions originales ?

Aurons-nous besoin de publier un calendrier hebdomadaire des tâches, comme dans nos propositions originales ?

Comme, une chronologie mise à jour? pensez-vous que ce serait utile pour vous deux?

Zoomez sur la carte en fonction de l'emplacement de l'utilisateur à l'aide de la géolocalisation. (Je me demande si cela est utile car les cartes sont affichées en fonction de l'emplacement entré par l'utilisateur dans les parcelles2.)

Je pense que l'idée ici était simplement que si vous avez entré un emplacement, nous devrions également essayer de le montrer à un niveau de zoom approprié, pas seulement le monde entier, centré sur votre emplacement. Lorsque les gens placent des emplacements, ils le font souvent à un niveau de résolution/de flou spécifique, car nous utilisons ce genre de bibliothèque sympa que nous avons développée : https://github.com/publiclab/leaflet-blurred-location#how -it- fonctionne -- ainsi, la précision des coordonnées lat/lon peut être utilisée pour indiquer un niveau de zoom.

Une autre question. Certaines tâches de la liste n'ont pas leurs propres problèmes. Leur créons-nous un problème avant de commencer à travailler sur ces tâches ?

Oui s'il vous plaît! Ce qui fonctionne généralement, c'est de créer d'abord un "problème de planification" de type liste avec des cases à cocher, et d'en diviser quelques-uns en leurs propres problèmes, que vous aborderez en premier. Ensuite, lancez d'autres problèmes distincts au fur et à mesure. Cela permet de ne pas créer un flot de problèmes d'un seul coup au début, et si les plans changent au fur et à mesure que nous en apprenons plus, nous n'avons pas à fermer un tas de problèmes, nous pouvons simplement ajuster le plan et les problèmes que nous avons créés afin loin. Merci, j'espère que cela a du sens!

Nos propositions originales avaient un calendrier avec des tâches prévues pour chaque semaine. C'est bon pour la responsabilité et les mentors qui s'assurent que nous sommes sur la bonne voie. Avec deux d'entre nous travaillant sur le même grand projet, préféreriez-vous que nous attribuions les tâches maintenant pendant que nous planifions, afin que nous sachions qui est en charge de quoi ? Ou préféreriez-vous que nous travaillions ensemble sur la liste, en effectuant la tâche suivante sur la liste ?

Cela dépend en partie de vous, en fonction de ce que vous préférez tous les deux. Nous n'avons pas de préférence marquée et nous accorderons suffisamment d'attention à la façon dont vous vous répartissez tous les deux le travail, nous ne sommes donc pas trop préoccupés par la responsabilité à cet égard. Si c'était moi, je diviserais les prochaines semaines de travail à une résolution assez élevée, afin que vous ne vous marchiez pas sur les pieds les uns des autres, mais que vous ayez simplement une idée plus générale de la division du travail plus loin. Et vérifiez simplement chaque semaine pour être sûr que vous avez suffisamment planifié les deux prochaines semaines.

Nous sommes une communauté très coopérative et bien que nous apprécions l'indépendance, nous sommes plus intéressés par la façon dont les membres de la communauté se soutiennent et s'appuient sur les compétences complémentaires et distinctes des uns et des autres. Les gens ont des préférences sur le type de travail que vous aimez faire, dans quoi vous êtes bon, dans quoi vous voulez vous améliorer et sur quoi vous aimez offrir un soutien (si vous connaissez très bien un ensemble de compétences). Ainsi, ces facteurs peuvent vous guider pour décider qui travaille sur quelles pièces.

Si vous constatez que vous avez besoin d'aide pour prendre ces décisions, les autres mentors et moi-même sommes heureux de vous donner des conseils ou des conseils. Alors tendez la main !

Merci, désolé c'était une longue réponse!

Enfin, je tiens à reconnaître que puisque le projet a progressé depuis que les propositions originales ont été faites, vous intégrez déjà de nouvelles tâches qui n'existaient pas dans vos plans originaux. Nous apprécions cela ! Mais je discute avec certains membres de la communauté de la possibilité de s'impliquer dans le codage de certaines parties de ceux-ci également, afin qu'il y ait suffisamment de capacité pour accueillir les nouvelles tâches. Aucune décision pour l'instant, mais j'espère que cela vous aidera à ne pas vous sentir dépassé !

Merci beaucoup @jywarren !

Aurons-nous besoin de publier un calendrier hebdomadaire des tâches, comme dans nos propositions originales ?

Comme, une chronologie mise à jour? pensez-vous que ce serait utile pour vous deux?

Je ne pense pas que cela nous importe de toute façon. Avoir une grande liste et simplement parcourir les éléments fonctionne tout aussi bien, c'est ainsi que j'ai travaillé sur mon site de projet personnel... très longue liste d'éléments à faire et je les ai fait un à la fois. :) Je voulais juste savoir s'il y avait une exigence pour un calendrier.

J'aime votre idée de diviser quelques semaines de tâches, puis de vous enregistrer et de vous soutenir mutuellement au fur et à mesure que nous les parcourons.

Pining ce problème!

Salut @nstjean @crisner ,

Tout d'abord _félicitations_ pour avoir été sélectionné comme stagiaires de sensibilisation 🙌 (j'espère que je ne suis pas en retard pour souhaiter ).
Je voudrais me présenter officiellement car je travaillerais principalement avec vous en tant que mentor 🎈 . Je suis donc l'un des contributeurs de PL et je le fais depuis plus de 2 ans je pense. J'ai commencé en tant qu'étudiant GSoC et maintenant je suis le mentor de divers programmes open source, notamment GSoC, GCI et Outreachy. J'ai commencé le projet LEL pendant mon GSoC et je l'ai activement maintenu et j'aimerais beaucoup de votre aide pour ajouter des fonctionnalités plus impressionnantes dans LEL avec les conseils de @jywarren et @ebarry !!!

Merci!
Meilleur,
Sagarpreet

Merci @sagarpreet-chadha ! C'est tellement cool que vous ayez commencé le LEL pendant votre GSoC. :) J'ai hâte de travailler avec vous!

Merci, @sagarpreet-chadha ! J'ai hâte de travailler avec vos conseils et votre soutien continus! :)

Salut @crisner , @nstjean
Pouvez-vous les gars ajouter les relations publiques que vous souhaitez que je passe en revue ou débogue dans ce problème, je reçois beaucoup d'e-mails ces derniers temps et je peux donc manquer certaines notifications. Ce serait plus simple pour moi d'en venir à ce problème. Merci !

Merci, c'est aussi très bien pour la gestion globale du projet. Au fur et à mesure que nous réglons les problèmes, nous devons garder une bonne trace de ce que nous avons fait ou non. Cela peut être fait dans une liste de contrôle qui :

  • [ ] pouvez
  • [x] être
  • [ ] coché

Une autre façon (bien que je pense que le ping dans ce fil devrait également se produire) consiste à maintenir les cartes du projet GitHub. Mais je pense que je me souviens que vous aviez déjà mis en place un système de suivi de projet ? Peut-être que ma mémoire aujourd'hui me joue des tours 😄

Désolé juste pour clarifier, si vous utilisez la méthode des cases à cocher, cela n'envoie pas d'e-mail à Sagarpreet, c'est donc une bonne raison de laisser un commentaire AUSSI si vous allez utiliser la liste de cases à cocher que vous avez faite ci-dessus. Merci!

D'accord ça marche ! J'utiliserai les cases à cocher ci-dessus pour suivre ce qui est terminé
et non, nous pouvons laisser un commentaire avec un lien vers les problèmes afin que vous obteniez tous les deux
notifié. Je suppose que la modification de la liste ne va pas vous avertir ?

>
Oui, nous utilisons Notion pour coordonner et organiser nos problématiques !

Oh cool, super. Si vous souhaitez partager un lien qui est génial, mais pas
obligatoire! Merci!

Le mer. 11 déc. 2019 à 14:57 Natalie St Jean [email protected]
a écrit:

D'accord ça marche ! J'utiliserai les cases à cocher ci-dessus pour suivre ce qui est terminé
et non, nous pouvons laisser un commentaire avec un lien vers les problèmes afin que vous obteniez tous les deux
notifié. Je suppose que la modification de la liste ne va pas vous avertir ?

>
Oui, nous utilisons Notion pour coordonner et organiser nos problématiques !

-
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/6887?email_source=notifications&email_token=AAAF6J6IUAZV4OHXY3DSCFTQYFA2BA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN56
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6JYPGCXV2YXTBDXXADLQYFA2BANCNFSM4JTZOL5A
.

@sagarpreet-chadha, j'aimerais avoir vos retours sur ce PR #296 pour le numéro #252 sur LEL. Merci!

@sagarpreet-chadha PR #293 pour le numéro 6889.

PR #6957 pour #6946 - Inclus quelques idées/suggestions.

PR #300 pour le numéro #299 (Suite du PR #296 )

Oui en passant par les relations publiques maintenant.

Hey @crisner , j'ai laissé un petit changement dans votre RP. Après cela, nous pouvons fusionner ce PR dans LEL.
Quel problème envisagez-vous de faire après cela?

Salut @nstjean ,
Je pense que vous pouvez faire un petit changement que @jywarren a laissé dans https://github.com/publiclab/plots2/pull/6858 afin que vous puissiez continuer à travailler sur le problème # 6957 .

Merci à vous deux!!! Vous faites tous les deux des boulots incroyables 🙌 !!!

Merci, @sagarpreet-chadha ! Je travaillais sur le numéro 301 . Après cela, j'avais l'intention de travailler sur la création d'un nom de domaine en tant qu'option réglable par défaut sur les pages gh comme suggéré lors de l'utilisation de la fonctionnalité d'intégration. Je cherchais aussi à ajouter des tests.

Commentaires requis pour le PR #303 sur le problème #301

Commentaires nécessaires pour le PR #300 pour le problème #299 mis à jour avec les changements du PR #296 . Merci!

@sagarpreet-chadha, j'ai besoin de vos conseils sur les problèmes suivants que j'ai créés. Pourriez-vous y jeter un œil et me faire savoir si je dois ajouter ou supprimer quelque chose ? Voici les problèmes : numéro 304 , numéro 305 et
numéro #6978. Merci!

Salut tout le monde! Je sais que beaucoup de gens célèbrent les vacances dans les semaines à venir, sans parler du Nouvel An. Veuillez contacter nos fils de discussion par e-mail pour connaître votre disponibilité et l'heure à laquelle vous partirez (s'il vous plaît !)

@crisner , les problèmes de FTO ont l'air incroyables !!! La prochaine fois dans LEL, nous pourrons également utiliser le même modèle d'émission FTO de plots2. Merci!

Ok, j'ai configuré cela pour permettre la visualisation publique ! Espérons que cela fonctionne correctement. Ceci est notre planificateur de projet Notion.
https://www.notion.so/nstjean/df760cb3a7514788bf448ce6f6d2404a?v=77cad99bbb0c4151965fe98c47d6499c

@jywarren Pour l'élément " Rediriger /maps vers /map, laisser /archive en tant qu'archive" -- est-ce que /map existe encore ? Déplaçons-nous simplement sur /maps vers une autre URL ?

ça n'existe pas encore ! Je pense que @sagarpreet-chadha avait peut-être commencé le
ça, mais il nous faudra :

  1. une nouvelle route à /map/ et /map/:slug
  2. une nouvelle page avec une carte plein écran
  3. une variété de configuration de permalien pour afficher les emplacements corrects

@sagarpreet-chadha, peut-être pouvez-vous nous tenir au courant des dernières nouvelles à ce sujet afin que nous puissions
commencer à le diviser en plusieurs parties ? Si vous avez déjà commencé, nous pouvons peut-être
fusionner un sous-ensemble ? Merci!

Le lun. 16 déc. 2019 à 14:31 Natalie St Jean [email protected]
a écrit:

@jywarren https://github.com/jywarren Pour l'élément "Redirect /maps to
/map, laissez /archive en tant qu'archive" -- est-ce que /map existe déjà ? Sommes-nous en train de déménager
sur /maps vers une autre URL ?

-
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/6887?email_source=notifications&email_token=AAAF6J7PULTKWGWIZ75HQ7DQY7JSXA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXH52ZKTDN56
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6J75QMNSU7J7X362XI3QY7JSXANCNFSM4JTZOL5A
.

D'accord, super! Le rompre sera excellent. j'attends de voir ce qui a été
commencé, le cas échéant, et partez de là ! Je remarque beaucoup de choses sur le
La liste plots2 tourne autour du fonctionnement de la page de la carte.

>

Salut @jywarren , @nstjean
Il a été mis en place par moi 😄 !

J'avais implémenté une nouvelle route /map avec une carte en plein écran avec URL Hash implémentée. Vérifiez ce lien :

https://publiclab.org/map/mapknitter#lat =60.23981116999893&lon=5.976562500000001&zoom=4

Il y a un bug que vous pouvez voir lorsque vous ouvrez cette page, la partie inférieure de application.html.erb chevauche la carte (ce bug n'est pas reproductible sur local je suppose).

Dans PR https://github.com/publiclab/plots2/pull/6874 , j'avais implémenté cette nouvelle route /map avec une carte en plein écran et un hachage d'URL fonctionnel. Cette page a été implémentée afin qu'iFrame puisse être utilisé pour intégrer cette carte à d'autres sites Web. Merci!

Commentaires requis pour PR #307 pour le numéro #306 . Merci!

@sagarpreet-chadha, merci d'avoir examiné et approuvé le PR ci-dessus aujourd'hui. J'ai également besoin de commentaires pour PR #309 sur le problème #308 . Merci!

@crisner , les problèmes de FTO ont l'air incroyables !!! La prochaine fois dans LEL, nous pourrons également utiliser le même modèle d'émission FTO de plots2. Merci!

Merci, @sagarpreet-chadha ! Je peux modifier les problèmes pour utiliser le modèle dans plots2, si cela vous convient.

@sagarpreet-chadha Ça a l'air excellent !! Je crois que le pied de page est la partie qui se chevauche... il n'apparaît pas sur local car le contenu n'existe pas sur local. Je n'ai pas regardé si c'était un wiki ou autre chose. Je suis sûr que c'est facilement caché via css pour la page de la carte.

Génial @sagarpreet-chadha !!! Sortons un nouveau numéro avec les prochaines choses à faire à ce sujet. Super!

J'ai extrait un tas de https://github.com/publiclab/plots2/issues/6801 et les ai déplacés dans un problème plus spécifique ici : https://github.com/publiclab/plots2/issues/6993

J'espère que c'est utile; heureux de vous adapter si vous souhaitez vous organiser différemment. Je vais également entrer et cocher un certain nombre d'entre eux qui semblent être terminés à partir du #6801 !

Oui @nstjean , je pense que position:absolute devrait faire l'affaire. Merci!

PR #311 sur le numéro #310 prêt pour examen.

J'ai rencontré un petit problème ! Les modifications que j'ai apportées dans leaflet-blurred-location sont nécessaires pour que je puisse terminer les prochaines étapes dans PublicLab.editor et plots2 . Comment puis-je faire apparaître les modifications ? Existe-t-il un moyen de lier mes différents référentiels ? Avons-nous besoin de mettre à jour quelque chose dans LBL pour que les modifications s'affichent dans PL.editor ? Je ne sais pas trop comment m'y prendre !

Ah ! Ok donc nous devons incrémenter le numéro de version npm dans le package.json
fichier dans la bibliothèque, alors moi ou Sagarpreet devons publier la nouvelle version.
Ensuite, l'exécution de npm install ou npm update dans la bibliothèque en aval tirera
la nouvelle version ! La bibliothèque amont est-elle prête à publier ? Merci!!

Le mer. 18 déc. 2019, 14:05 Natalie St Jean [email protected]
a écrit:

J'ai rencontré un petit problème ! Les changements que j'ai apportés dans
brochure-blurred-location sont nécessaires pour que je puisse terminer les prochaines étapes dans
PublicLab.editor et plots2. Comment puis-je faire apparaître les modifications ? Est
existe-t-il un moyen de lier mes différents référentiels ? Avons-nous besoin de mettre à jour
quelque chose dans LBL pour que les changements apparaissent dans PL.editor ? je ne suis pas vraiment sûr
comment s'y prendre !

-
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/6887?email_source=notifications&email_token=AAAF6JZWENGUCIJTSV7OJD3QZJX6TA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXWWH2HJKT ,
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6JYBNTLQQ5DQDLYEHK3QZJX6TANCNFSM4JTZOL5A
.

Ah ! D'accord! C'est la dernière grande chose dont j'ai besoin pour être publiée pour ensuite apporter des modifications dans les autres dépôts : https://github.com/publiclab/leaflet-blurred-location/pull/224

Fantastique. Et, il y a un ensemble de règles sur semver.org pour la façon dont nous gérons
numéros de version ; la version courte est qu'une correction de bogue déclencherait un
augmentation de +0.0.1, alors qu'une nouvelle fonctionnalité est un changement de version "mineur" +0.1.0,
et si nous devions apporter un changement radical dont les utilisateurs en aval devraient
refactoriser avant la mise à jour, ce serait une version "majeure" de +1.0.0.

C'est un système sympa pour coordonner les versions entre les bibliothèques ! l'espère
logique!!!

Le mer. 18 déc. 2019, 14:28 Natalie St Jean [email protected]
a écrit:

Ah ! D'accord! C'est la dernière grande chose que j'ai besoin d'être publié pour ensuite faire
changements dans les autres dépôts : publiclab/leaflet-blurred-location#224
https://github.com/publiclab/leaflet-blurred-location/pull/224

-
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/6887?email_source=notifications&email_token=AAAF6J4YBGMZH6MNAGR7SLLQZJ2ELA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVX5WWY2ZLOBE
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6J5CTEY22HNNAYZNHJ3QZJ2ELANCNFSM4JTZOL5A
.

Ahhh, donc cela serait considéré comme une fonctionnalité "mineure" ?

Oui c'est correct!

Le mer. 18 déc. 2019, 15h00 Natalie St Jean [email protected]
a écrit:

Ahhh, donc cela serait considéré comme une fonctionnalité "mineure" ?

-
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/6887?email_source=notifications&email_token=AAAF6JZDFSG2WTIIX5ZOKNDQZJ54TA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMV2216ZKTDN5EWW2HJJKTDN5WWE
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6J73ZJL26U6MH6X4YWDQZJ54TANCNFSM4JTZOL5A
.

J'ai mis à jour la version, corrigé l'erreur qui se produisait et ajouté des tests. C'est prêt à fusionner ! https://github.com/publiclab/leaflet-blurred-location/pull/224

Commentaires requis pour le PR #314 sur le problème #313

Commentaires nécessaires pour les relations publiques https://github.com/publiclab/plots2/pull/7020 pour le problème https://github.com/publiclab/plots2/issues/7019

Commentaires nécessaires pour les relations publiques https://github.com/publiclab/PublicLab.Editor/pull/374 pour le problème https://github.com/publiclab/PublicLab.Editor/issues/373

PR https://github.com/publiclab/plots2/pull/6957 a été mis à jour avec les modifications apportées à LBL et devrait être prêt à fusionner maintenant.

J'ai mis à jour la liste ci-dessus avec les PR de cette semaine. ??

@sagarpreet-chadha, le PR pour le numéro 6978 montre des erreurs car la LIE n'est pas mise à jour dans plots2. Dois-je mettre à jour la version npm avec « +0.1.0 » comme indiqué dans la conversation ci-dessus ou devrions-nous la mettre en attente pour le moment ?

PR #315 pour le numéro #297 prêt pour examen.

En travaillant sur le problème #297 , j'ai trouvé que la documentation de la couche windrose n'était pas claire ici :
https://github.com/publiclab/leaflet-environmental-layers#to -use-wind-rose-layer-

Je me demandais s'il serait préférable pour plus de clarté de l'ajouter ici comme première étape avant l'initialisation ? :
https://github.com/publiclab/leaflet-environmental-layers#real -time-layers-

Besoin de révision/fusion pour le problème https://github.com/publiclab/leaflet-blurred-location/issues/225 PR https://github.com/publiclab/leaflet-blurred-location/pull/226

J'ai également posté quelques idées pour le modal popup https://github.com/publiclab/plots2/issues/7031 à la recherche de commentaires !

Je vais essayer de répondre un peu plus aujourd'hui même si je me lance en vacances
mode de voyage :-P désolé !

Concernant npm version bump, oui ! Ce serait génial.

Et si vous avez un PR prêt pour la fusion, veuillez le noter dans les commentaires sous
le PR et après avoir examiné, je peux aller de l'avant et le faire ! Merci!

Le lundi 23 décembre 2019 à 10h27 Natalie St Jean [email protected]
a écrit:

Besoin d'examen/fusion pour le problème publiclab/leaflet-blurred-location#225
https://github.com/publiclab/leaflet-blurred-location/issues/225 PR
publiclab/leaflet-blurred-location#226
https://github.com/publiclab/leaflet-blurred-location/pull/226

J'ai aussi posté quelques idées pour le modal popup #7031
https://github.com/publiclab/plots2/issues/7031 à la recherche de commentaires !

-
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/6887?email_source=notifications&email_token=AAAF6J4J3OFPHRRUPHIJ7VDQ2DKGHA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMV6ZHJ850YLDTDN5 ,
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6J3QIYDCSGPHX3I7M3DQ2DKGHANCNFSM4JTZOL5A
.

Pas de soucis! Profitez de votre temps libre !! Je serai déconnecté les deux prochains jours avec les fêtes de fin d'année aussi.

PR #317 pour le numéro #316 prêt pour examen. J'ai cogné la version dans celui-ci.

Je vais essayer de répondre un peu plus aujourd'hui même si je me lance en vacances
mode de déplacement :-P

Aucun problème! Prends ton temps. Profitez de vos vacances! S'amuser!

Je me demandais s'il serait préférable pour plus de clarté de l'ajouter ici comme première étape avant l'initialisation ? :

Oui c'est logique !

J'ai passé en revue tous les PR :)

Besoin d'avis pour ce PR #320 . C'est encore un travail en cours, mais j'aimerais avoir des commentaires à ce sujet. Merci!

J'ai apporté des modifications à PR https://github.com/publiclab/leaflet-blurred-location/pull/226 :)

Hey @nstjean , j'ai fusionné le PR ci-dessus :) sur quoi travailles-tu maintenant ?

@crisner , vous pouvez continuer à travailler sur la barre de contrôle dans LEL. Nous pouvons demander à @jywarren de commenter plus tard si nous voulons ou non une nouvelle interface utilisateur pour les petites cartes.

Merci à tous les deux!!! ??

Excellent! Ensuite, je vais parcourir la liste sur # 6993 - je corrige d'abord l'erreur de carte, puis j'affiche correctement le pied de page.

Correction de la route/des valeurs par défaut pour /map/ - Problème https://github.com/publiclab/plots2/issues/7045 PR https://github.com/publiclab/plots2/pull/7046 - prêt pour examen/fusion

WIP, poussez le pied de page sous la carte dans /map/ - Issue https://github.com/publiclab/plots2/issues/7047 PR https://github.com/publiclab/plots2/pull/7048

Besoin d'informations sur ce bogue, la carte permet de faire défiler les limites et enregistre une longitude incorrecte : https://github.com/publiclab/plots2/issues/7052

Publication d'un FTO dans LEL, numéro #322

PR #319 pour FTO issue #305 prêt.

PR #320 prêt pour examen.

En attendant, je prévois de travailler sur l'écriture de tests pour le même. Les autres tâches sur lesquelles j'avais prévu de travailler après cela sont,

  • Afficher la marque "nouveaux éléments" dans le menu Calques lorsque vous entrez dans une zone avec un nouveau calque #133
  • Suivez la zone de délimitation de chaque couche et affichez uniquement celles qui se croisent avec la vue actuelle n ° 117 - je pense que cela est abordé dans le PR ci-dessus.
  • La recherche n'est pas ouverte par défaut mais derrière une icône en forme de loupe qui s'ouvre
  • Recherche de localisation : utilisez le décodeur de saisie semi-automatique et recentrez la carte

J'ai besoin d'informations sur #7048

Arrêtez de faire défiler les limites : numéro 7052 PR #7058

Créez des routes /map/wiki_slug qui source lat/lon/zoom à partir de la page wiki correspondante - Problème #7060 PR #7061 - WIP

Ensuite :

  • Ajouter le bouton « Afficher la carte » aux barres latérales des pages géolocalisées, en ouvrant /map/SLUG pour les wikis et /map#zoom/lat/lon permalink pour les notes
  • obtenir des notes, des wikis et des personnes affichées sur les pages /map/SLUG
  • Le curseur dans LBL ne se met pas à jour lorsque la carte est utilisée pour zoomer et dézoomer
  • Les permaliens sur /map/ devraient afficher l'emplacement actuel dans le hachage d'URL même en utilisant les données par défaut

Bonne année à tous :)

J'ai examiné les PR ci-dessus. Merci!

Pour une raison quelconque, je ne parviens pas à instancier le nouveau contrôle de couche dans Jasmine. Il semble y avoir un problème avec les classes étendues L.Control.Layers. Les classes Leaflet étendues à partir de L.Control (ex. embedControl) semblent accessibles dans les fichiers de test. J'ai essayé de configurer le karma avec jasmin et cela semble fonctionner, mais j'ai des problèmes avec le chargement des appareils. Je vais réessayer et partager mes relations publiques si je suis toujours bloqué.

J'ai besoin de quelques éclaircissements dans les tâches suivantes :

  • Suivre le cadre de délimitation de chaque calque et afficher uniquement ceux qui se croisent avec la vue actuelle #117
  • Afficher la marque "nouveaux éléments" dans le menu Calques lorsque vous entrez dans une zone avec un nouveau calque #133

Je vais travailler sur le contrôle de recherche en attendant.

À revoir : ajouter un remplacement de zoom aux cartes dans la barre latérale et en haut de la page : problème #7077 PR #7078

7048 est prêt à fusionner

7046 devrait être prêt à fusionner, les modifications demandées sont apportées dans #7061

J'ai quelques questions dans https://github.com/publiclab/plots2/issues/7052

Ces deux objets ont été ajoutés au #7061 :

  • Ajouter le bouton « Afficher la carte » aux barres latérales des pages géolocalisées, en ouvrant /map/SLUG pour les wikis et /map#zoom/lat/lon permalink pour les notes
  • Les permaliens sur /map/ devraient afficher l'emplacement actuel dans le hachage d'URL même en utilisant les données par défaut

J'ai besoin d'aide pour le PR #328 concernant le test de layerBrowser.js . Merci!

PR #326 pour le numéro #325 prêt pour examen.

Liste ci-dessus mise à jour avec les PR.

PR #329 prêt pour le numéro #327

Je travaillerai ensuite sur le numéro 87 .

Taguez-moi sur Gitter si vous avez besoin d'aide !

Le lundi 6 janvier 2020, 20h47 Jeffrey Warren [email protected]
a écrit:

Impressionnant!!!

Le lun. 6 janvier 2020 à 09:14 Renisha Christie. Un <
[email protected]>
a écrit:

Je vais travailler sur le problème #87
https://github.com/publiclab/leaflet-environmental-layers/issues/87
Suivant.

-
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/6887?email_source=notifications&email_token=AAAF6J2AVCAIVW42QZSMEPLQ4M4GDA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVX2HJKTDN5WW2HJJKTDN5WWK3TUL52HS4DFVREXG43VMVBW63LNMVX2HJKTDN5WW
,
ou se désinscrire
<
https://github.com/notifications/unsubscribe-auth/AAAF6JZRV6YNMI7MIMYANKLQ4M4GDANCNFSM4JTZOL5A

.

-
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/6887?email_source=notifications&email_token=AJXHQZ7ICNM4G7P6Z23NR7TQ4NDP3A5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNEI771A5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNEI71VXWWHJDNGO
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AJXHQZ6ERNXZ64573UNHJMLQ4NDP3ANCNFSM4JTZOL5A
.

Bien sûr @ IshaGupta18 ! Merci!

J'ai WIP https://github.com/publiclab/plots2/pull/7125 - pas fait, mais je peux donner mon avis !

https://github.com/publiclab/plots2/pull/7061 me frustre. J'ai corrigé le test qui échouait, mais il y a des erreurs de synchronisation sur tous mes tests de cartes.

https://github.com/publiclab/plots2/pull/6957 est prêt à fusionner

https://github.com/publiclab/plots2/pull/7105 a été modifié, devrait être configuré pour fusionner

J'ai créé deux problèmes qui ne sont pas des problèmes trop difficiles pour quiconque cherche un petit problème à résoudre.
https://github.com/publiclab/plots2/issues/7122
https://github.com/publiclab/plots2/issues/7120 - Eh bien, celui-ci, je ne suis pas sûr. Cela pourrait finir par être plus compliqué, mais j'ai le sentiment que c'est quelque chose de stupide. Si personne d'autre ne montre d'intérêt, je le découvrirai.

La prochaine étape sera l'ajout de notes, de wikis et de personnes sur la /map/ avec des popups.

Merci @IshaGupta18 !

@jywarren , entrées nécessaires pour PR #330 . Merci!

Le PR #330 pour le numéro 87 est prêt. Merci!

  • PR #332 prêt pour examen
  • Création d'un FTO avec le problème #333

PR #340 pour le numéro #338 prêt pour examen

@jywarren , s'il vous plaît jeter un oeil à PR # 7157

PR https://github.com/publiclab/plots2/pull/7159 est prêt à être révisé et fusionné

PR https://github.com/publiclab/plots2/pull/7061 ne passera tout simplement pas travis sur le serveur avec les tests de système de carte que j'ai écrits, bien qu'il passe sur mon environnement local. Cess jette un œil.

PR https://github.com/publiclab/plots2/pull/7125 est toujours un travail en cours. :)

Problème https://github.com/publiclab/plots2/issues/7084 J'ai besoin de commentaires avant de commencer.

Publication du numéro de FTO #7217. Cela résoudrait les problèmes 404 à partir du code d'intégration copié à partir de https://publiclab.org/map/

  • Besoin de contributions pour le numéro 7220 et le numéro 344
  • PR #343 prêt pour examen

Suivant sur ma liste :

  • Implémenter le mode minimal #123

@nstjean , je vous ai laissé un commentaire ici : #6969. Pourriez-vous y jeter un œil ? Merci!

PR #345 pour le numéro #337 prêt pour examen.

@crisner , celui-ci est intéressant à résoudre https://github.com/publiclab/leaflet-environmental-layers/issues/123 , nous pouvons discuter de l'approche avant de passer au code. Qu'est-ce que tu penses?

@nstjean ... Avant que @jywarren ne https://github.com/publiclab/plots2/pull/6969 .

Bon travail :)
Merci!

@sagarpreet-chadha J'ai apporté les modifications demandées dans ce PR avant de fusionner... est-ce que j'ai raté quelque chose qui n'a pas été modifié ?

@crisner , celui-ci est intéressant pour résoudre publiclab/leaflet-environmental-layers#123 , nous pouvons discuter de l'approche avant de passer au code. Qu'est-ce que tu penses?

Bien sûr, @sagarpreet-chadha ! Je pensais que nous devions vérifier quel type de marqueur serait "léger" sur la page. Je me demandais s'il ne serait pas préférable de changer tous les marqueurs en forme de dépliant, par exemple le circleMarker ou comme ce que Jeff a utilisé dans unearthing-pvd, je pense que cela a été fait en utilisant glify. Il faudrait ajouter l'icône du mode minimal pour toutes les couches et les rendre en fonction du mode que l'utilisateur a défini sur la carte. Avez-vous des idées que vous aimeriez partager?

Je voulais également suggérer que le nouveau menu sur lequel travaille

https://stable.publiclab.org/map

... avec toutes les couches disponibles. Ce serait vraiment un bel endroit pour présenter ce nouveau travail! Qu'est-ce que tu penses?

Oui! Ce serait super excitant !

PR #346 pour le numéro #344 prêt pour examen

Besoin de contributions sur le problème #347

https://github.com/publiclab/leaflet-blurred-location/pull/232 est prêt pour la fusion.

Et une fois que cela sera fusionné et publié, je pourrai l'utiliser pour terminer https://github.com/publiclab/plots2/pull/7125

Merci a tous! En passant par certains maintenant!

Le mer. 15 janv. 2020 à 9h44 Natalie St Jean [email protected]
a écrit:

7189 https://github.com/publiclab/plots2/pull/7189 est prêt à fusionner

7222 https://github.com/publiclab/plots2/pull/7222 est prêt à fusionner

-
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/6887?email_source=notifications&email_token=AAAF6J7PRXBZNJTBXW6TST3Q54N37A5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVAQ3WWZLOJK93 ,
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6J3MTKBV2XYKVVB5YYTQ54N37ANCNFSM4JTZOL5A
.

Besoin de commentaires sur PR #352 pour le problème #349 .

Concernant les popups qui ne s'affichent pas sur les marqueurs :

Si nous changeons cette ligne popup = new Popup(options).setContent(popup); en popup = new Popup(options).setContent(popup._content ? popup._content : popup); dans la ligne de repo du dépliant :

https://github.com/Leaflet/Leaflet/blob/d1a1e97b8290f642eb677284af53c2db64199a76/src/layer/Popup.js#L320

Tout fonctionne bien alors !

Une façon de modifier cette ligne consiste à utiliser https://www.npmjs.com/package/patch-package , cela modifie cette ligne dans le dossier des modules de nœud après l'installation du fil. L'avantage est que nous pouvons toujours mettre à jour la version du dépliant s'il y en a (nous deviendrons en quelque sorte un utilisateur statique du dépliant si nous forkons le dépliant et faisons ce changement).

Bonne nuit les gars!

oh mon Dieu, j'ai totalement mal compris, s'il vous plaît pardonnez-moi @sagarpreet-chadha ! C'est un bug de Leaflet ? Je suppose que je préférerais bifurquer Leaflet et le réparer et pointer vers notre branche, mais est-ce un problème connu, qui pourrait être corrigé par eux bientôt? Ou, y a-t-il un moyen pour nous de remplacer cette ligne dans notre code, après avoir inclus Leaflet ?

@crisner @nstjean qu'en pensez-vous ? Je suppose que nous ne devrions pas retarder la publication de PublicLab.org si nous n'avons pas encore de chemin rapide pour résoudre ce problème. J'ouvre un sujet où nous pourrons en discuter un peu plus, j'adorerais avoir de vos nouvelles. Merci!

Pouvons-nous créer une pull request sur Leaflet pour le correctif afin qu'il soit transmis par eux ? Je suis curieux, les popups fonctionnaient-ils et maintenant ils ne fonctionnent plus ? Ou était-ce un problème récurrent ?

Je pense que nous devons rassembler un peu plus d'informations sur l'État avant et
l'état maintenant, et quand c'est arrivé, et rassemblez-le dans le nouveau numéro que je
fait, avant de prendre une décision finale. Merci!

Le ven. 17 janv. 2020 à 14h15 Natalie St Jean [email protected]
a écrit:

Pouvons-nous créer une pull request sur Leaflet pour le correctif afin qu'il soit poussé
par eux ? Je suis curieux, est-ce que les popups fonctionnaient et maintenant ils
non ? Ou était-ce un problème récurrent ?

-
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/6887?email_source=notifications&email_token=AAAF6JZADIQOCNTMHKDZPZDQ6H7BVA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNIVment-5WW2CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNIVment-5WW2ZUKTDN45
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/AAAF6JYUHJ26E5O2XWMB57DQ6H7BVANCNFSM4JTZOL5A
.

Le PR #353 pour le numéro #347 est prêt.

Je suis tellement excitée qu'hier j'ai eu du succès avec les tests Jasmine ! Ce furent de longues journées d'essais et d'échecs répétés.
https://github.com/publiclab/leaflet-blurred-location/pull/235 est le correctif de LBL
https://github.com/publiclab/leaflet-blurred-location-display/pull/93 est le correctif de LBLD - peut ne pas être nécessaire
Et aujourd'hui, je vais examiner les échecs des tests de LEL et de la couche People. Je parie que c'est lié.

Pas encore prêt, mais j'aimerais avoir des commentaires sur le PR #356 pour le numéro 123 . Merci!

J'ai ajouté un correctif pour LEL https://github.com/publiclab/leaflet-environmental-layers/pull/358 - pourrait ne pas être nécessaire

Commenté/revu ci-dessus PR's :)

Je pourrais également utiliser les commentaires ici : https://github.com/publiclab/leaflet-blurred-location/issues/233#issuecomment-576752535

Demande de modifications apportées au PR #353 pour le numéro 347 et est prêt à être examiné.

Je voudrais aussi des contributions sur PR #356

Révisé ci-dessus PR. Merci!

Le jeu. 23 janv. 2020, 01:28 Natalie St Jean, [email protected]
a écrit:

Question sur la page de la carte ici : #6993 (commentaire)
https://github.com/publiclab/plots2/issues/6993#issuecomment-577352894

Ajout de PLpeopleLayer à LEL : publiclab/leaflet-environmental-layers#361
https://github.com/publiclab/leaflet-environmental-layers/pull/361 [WIP]

publiclab/leaflet-blurred-location#235
https://github.com/publiclab/leaflet-blurred-location/pull/235 prêt
pour fusionner

-
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/6887?email_source=notifications&email_token=ADSCRRPLFK5SMXGK32SZVBDQ7CQOBA5CNFSM4JTZOL5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVX23ZHJKJUP212
ou se désinscrire
https://github.com/notifications/unsubscribe-auth/ADSCRRPCX2YPMQEJCPB4GGDQ7CQOBANCNFSM4JTZOL5A
.

PR #356 pour le numéro 123 prêt pour examen. Merci!

publiclab/leaflet-blurred-location#235 - prêt pour la fusion

https://github.com/publiclab/leaflet-environmental-layers/pull/361 - prêt pour la fusion

https://github.com/publiclab/plots2/pull/7358 - devrait être prêt, travis a échoué et doit être redémarré

Souhaitez-vous des commentaires en LEL :
https://github.com/publiclab/leaflet-environmental-layers/issues/364
https://github.com/publiclab/leaflet-environmental-layers/issues/365

Il semble qu'une erreur se produise sur certaines pages du wiki - cela ressemble à celles avec une carte en ligne. Cela empêche l'utilisation du bouton "ajouter un emplacement". Je l'examinerai plus en profondeur demain : https://github.com/publiclab/plots2/issues/7346

Commentaires requis pour PR #367

Merci a tous! Regardé à travers ceux-ci !

https://github.com/publiclab/leaflet-environmental-layers/pull/368 - supprime LBL et également le numéro de version de LEL afin que les modifications puissent être appliquées à plots2. Prêt pour la fusion.

Les PR suivants sont prêts à être examinés et fusionnés

PR #380 pour le numéro 379 prêt pour examen.

https://github.com/publiclab/plots2/pull/7425 prêt pour examen et fusion

Je vais clore ça puisque le processus est passé au #7432 !!! Merci a tous!

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