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

feat(annuaire): Improve psy views #34

Merged
merged 25 commits into from
Feb 8, 2022
Merged

feat(annuaire): Improve psy views #34

merged 25 commits into from
Feb 8, 2022

Conversation

desoindx
Copy link
Contributor

No description provided.

@desoindx desoindx marked this pull request as draft January 22, 2022 16:22
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 22, 2022 16:22 Inactive
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 22, 2022 16:39 Inactive
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 22, 2022 16:50 Inactive
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 22, 2022 17:00 Inactive
@desoindx desoindx force-pushed the feat/psy-file branch 3 times, most recently from 713ff3b to 716d8ce Compare January 23, 2022 14:28
@desoindx desoindx changed the base branch from main to feat/annuaire January 23, 2022 14:33
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 23, 2022 14:36 Inactive
Base automatically changed from feat/annuaire to main January 24, 2022 08:27
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 24, 2022 17:04 Inactive
@desoindx desoindx marked this pull request as ready for review January 26, 2022 07:41
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g January 27, 2022 08:38 Inactive
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g February 7, 2022 08:49 Inactive
@@ -0,0 +1,6 @@
/* @preserve
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pourquoi on fait une ref sur la lib dans node_modules ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

j'ai pas compris la question

const [options, setOptions] = useState(AROUND_ME_OPTION);

const searchCommunes = () => {
if (filterText.length > 2 && filterText !== AROUND_ME) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pourquoi supérieur à 2

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pour eviter les recherches inutiles

@@ -8,7 +8,7 @@ export default (sequelize) => {
address: DataTypes.STRING,
archived: DataTypes.BOOLEAN,
cdsmsp: DataTypes.STRING,
coordinates: DataTypes.GEOMETRY("POINT"),
coordinates: DataTypes.GEOMETRY("POINT", 4326),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'est quoi cette valeur

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

c'est le SRID, on en a deja parlé lors de la création de la table. On a besoin d'un id de reference (celui la est l'ISO) partout pour bien calculer les distances

src/components/Directory/Psychologist.tsx Show resolved Hide resolved
src/components/Directory/PsychologistsMap.tsx Show resolved Hide resolved
Copy link
Contributor

@carolineBda carolineBda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tu préfère rajouter les tests plus tard ?

@desoindx
Copy link
Contributor Author

desoindx commented Feb 7, 2022

tu préfère rajouter les tests plus tard ?

Yes ! Comme vu la derniere fois cette partie va beaucuoup changer je pense

@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g February 7, 2022 09:35 Inactive
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g February 7, 2022 09:41 Inactive
@github-actions github-actions bot temporarily deployed to mon-psy-sante-feat-psy-file-59yr5g February 7, 2022 09:55 Inactive
Copy link
Contributor

@carolineBda carolineBda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@github-actions
Copy link

github-actions bot commented Feb 8, 2022

🎉 Deployment for commit 324bc2f :

Ingresses
Docker images
  • 📦 docker pull ghcr.io/socialgouv/docker/wait-for-postgres:6.56.1
  • 📦 docker pull ghcr.io/socialgouv/mon-psy-sante/app:sha-324bc2f8d1fb1de35308d194e52254d9a80a2d17
Debug

@desoindx desoindx merged commit 0da2d20 into main Feb 8, 2022
@desoindx desoindx deleted the feat/psy-file branch February 8, 2022 10:16
SocialGroovyBot added a commit that referenced this pull request Feb 8, 2022
# [1.10.0](v1.9.5...v1.10.0) (2022-02-08)

### Features

* **annuaire:** Improve psy views ([#34](#34)) ([0da2d20](0da2d20)), closes [#28](#28)
@SocialGroovyBot
Copy link
Member

🎉 This PR is included in version 1.10.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants