Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pharmacy search function added. #5

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

pharmacy search function added. #5

wants to merge 5 commits into from

Conversation

aristideman
Copy link
Collaborator

Bonjour Pierre!
Prière de jeter un œil au travail que j'ai réalisé dans ma branche.
J'ai discuté avec Steve et Trésor pour leur demander l'état d'avancement de cette partie et ils devraient se voir afin de discuter.
De mon côté, j'ai tiré une branche comme convenu, j'ai cloné le travail qu'ils avaient commencé et j'ai modifié un peu la présentation de l'interface qu'ils avaient fait.
En bref, j'ai ajouté la fonctionnalité de recherche de pharmacies qui marche partiellement, car il faut revoir le css pour la prise en compte des rotations (landscape mode)

Future release :

  • Recherche en prenant en compte le second critère
  • Recherche de la pharmacie la plus proche par calcul des distances et comparaison
  • Amélioration du css pour le mode landscape et autre

@PrinceYoulou
Copy link
Member

PrinceYoulou commented Apr 26, 2016

Salut @aristideman ,

Good job!!!
Pour la partie adresse pharmacie je veux que tu puisse indexer les communes de Brazzaville pour commencer. Je suis en train de penser à un menu déroulant pour éviter les érreur de l'utilisateur, @pierrepo qu'est ce tu en penses?

Pour l'instant, ajoute l'option des communes juste.

Encore Bravo.

@pierrepo
Copy link
Contributor

pierrepo commented Apr 26, 2016

Salut @aristideman.
Excellent travail !
Je souhaite que tu fasses plutôt un pull request sur le dépot :
https://github.com/fongwama/EduPalu
Attention, il ne faut pas casser l'application existante.
Pour cela, peux-tu s'il te plait :

  • mettre toutes les fonctions JS dans js/places.js
  • mettre tout le css dans : css/places_style.css
  • mettre la base de données des pharmacies dans : data/places_db.json
  • renommer index.html en places.html
  • ne pas inclure core.js ni retina.js ni jquery (déjà présent dans l'application)
  • ne pas inclure json.js car on ne fait les requêtes qu'avec jsonquery.

@aristideman
Copy link
Collaborator Author

Bonsoir Pierre,
Merci pour ton feedback.
Je prends en compte toute tes suggestion et je reviendrai vers toi;

Merci encore.

Aristide

@aristideman
Copy link
Collaborator Author

Ok @PrinceYoulou , ça va être fait.

@++

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants