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

Dev login & upload & register page #55

Open
AntoineBruge opened this issue Jan 15, 2021 · 7 comments
Open

Dev login & upload & register page #55

AntoineBruge opened this issue Jan 15, 2021 · 7 comments
Labels
feature help wanted Extra attention is needed

Comments

@AntoineBruge
Copy link
Contributor

AntoineBruge commented Jan 15, 2021

Dev login page and upload page

A user can label images without login in.
A user can upload images but he has to login in first.

Connect to API : https://github.com/surfriderfoundationeurope/po-mobile-backend
A postman is now available

@AntoineBruge AntoineBruge added help wanted Extra attention is needed feature labels Jan 15, 2021
@jeanremy jeanremy changed the title Dev login page Dev login & upload page Jul 23, 2021
@jeanremy jeanremy changed the title Dev login & upload page Dev login & upload & register page Jul 23, 2021
@bertrandlalo
Copy link
Collaborator

How do we register/create account ?

@AntoineBruge
Copy link
Contributor Author

@bertrandlalo c'est une partie du travail qui reste à faire. Le login ne sert à rien pour l'instant. Il servira à uploader des images dans un futur proche. Les comptes créés depuis l'app peuvent être utilisés sur trashroulette.

@AntoineBruge
Copy link
Contributor Author

@jeanremy est sur le coup mais si ça t'intéresse d'y jeter un œil et de lui donner un coup de main aucun soucis @bertrandlalo

@jeanremy
Copy link
Collaborator

jeanremy commented Feb 11, 2022

Hello ! Petit point sur la mise en place du login et upload.

  • Mise en place du login
  • Mise en place de la page upload. On a le choix entre:
    • Reprendre l'écran d'upload qui existait et qu'on voit ici
    • Refaire un nouvel écran avec un input type file.

Après un rapide tour de l'existant, il se trouve que reprendre l'ancien cide apporte pas mal de complexité juste pour l'upload d'un fichier, là où un simple input file suffirait. De plus, pas mal d'erreurs TS sont lancés, on passera plus de temps à tout régler qu'à refaire un upload.

Pour tout ça, on peut partir de la branche upload-comeback. Sur cette branche, il reste à

  • Rajouter la view Upload (@jeanremy ) (en cours)
  • Rajouter le formulaire d'envoi de photo (@jeanremy ). J'ai envoyé un mail à @cmaneu pour connaitre le point d'api à utiliser pour envoyer une photo, et quel format est attendu. Pour ce formulaire, on peut faire simple et ajouter par la suite une preview de l'image que l'on souhaite envoyer, voir des loaders, etc. Pour la première version, je pense qu'il est mieux de faire du fonctionnel.
  • Tester l'upload

@jeanremy
Copy link
Collaborator

jeanremy commented Feb 13, 2022

@AntoineBruge Penses-tu qu'il faille ajouter un texte d'intro au formulaire d'upload ? Pour l'instant, j'ai juste ça, mais c'est un peu vide...
Capture d’écran 2022-02-13 à 21 52 44
?

@AntoineBruge
Copy link
Contributor Author

Hello @jeanremy ,

Désolé je viens de voir ton message. Je n'ai pas reçu de notif de Github. Bizarre.
Pour le moment ça me semble suffisant. Est-ce qu'on peut uploader plusieurs images à la fois avec cette méthode?
Comment arrive-t-on sur cette page? Via un onglet en haut à côté de "Tutorial"?

@zorafa m'a dit qu'il avait un soucis pour lancer le front depuis cette branche. Tu as une idée de pourquoi?

@jeanremy
Copy link
Collaborator

On peut tout à faire en charger plusieurs. Et on arrive sur cette page via une entrée de menu Upload, qui remplace Login (pas visible sur ma capture d'écran)

Aucune idée en revanche de pourquoi ça ne se lance pas :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

3 participants