Création du module DolibarrClient avec ses différentes routes & création du module WarehouseGUI pour le LCD M5Stack #7

Merged
Clement merged 9 commits from feat/client_dolibarr into develop 2023-11-10 15:47:41 +00:00
Member
No description provided.
Nicolas added this to the Follow-up #2 milestone 2023-09-29 14:02:51 +00:00
Nicolas added the
Kind/Feature
Scope
IoT
labels 2023-09-29 14:02:51 +00:00
Nicolas added 1 commit 2023-09-29 14:02:53 +00:00
Nicolas requested review from Clement 2023-09-29 14:05:26 +00:00
Clement added 1 commit 2023-11-09 14:56:44 +00:00
Clement added 1 commit 2023-11-09 14:58:41 +00:00
Clement added 1 commit 2023-11-09 15:11:59 +00:00
Clement requested changes 2023-11-10 10:11:50 +00:00
Clement left a comment
Owner

il y a quelque changement a faire
l'interface graphique intégré mériterai sa propre branche nan ?

il y a quelque changement a faire l'interface graphique intégré mériterai sa propre branche nan ?
config.ini Outdated
@ -10,6 +10,7 @@ monitor_speed = 115200
; notworthy ones:
; __PLATFORMIO_BUILD_DEBUG__ = debug mode
build_flags =
-std=c++17
Owner

vraiment utile @Nicolas ?

vraiment utile @Nicolas ?
Nicolas marked this conversation as resolved
@ -1,7 +1,10 @@
#ifndef PROGRAM_H
#define PROGRAM_H
#define DEFAULT_BAUD_RATE 115200
Owner

deja inclue dans les config

deja inclue dans les config
Nicolas marked this conversation as resolved
@ -0,0 +3,4 @@
#include <ArduinoJson.h>
#include <iostream>
#define WAITING_WIFI_DELAY 1000
Owner

mettre en config

mettre en config
Nicolas marked this conversation as resolved
@ -0,0 +48,4 @@
return -1;
}
std::string replace_id(const char *str, const char *id) {
Owner

utiliser les string d'Arduino

utiliser les string d'Arduino
Nicolas marked this conversation as resolved
@ -0,0 +126,4 @@
return -1;
}
int DolibarrClient::initialize_wifi() const {
Owner

faut pas faire sa dans le prog principal ?

faut pas faire sa dans le prog principal ?
Nicolas marked this conversation as resolved
@ -0,0 +10,4 @@
constexpr u_int32_t CREATED_RESPONSE = 201;
constexpr u_int32_t NO_CONTENT_RESPONSE = 204;
constexpr u_int32_t LOGIN_JSON_SIZE = 4096;
Owner

tout les constexpr les mettre dans les fichiers ini (env et config)

tout les constexpr les mettre dans les fichiers ini (env et config)
Nicolas marked this conversation as resolved
@ -0,0 +1,5 @@
//
Owner

c'est quoi se fichier ?

c'est quoi se fichier ?
Nicolas marked this conversation as resolved
@ -0,0 +1,40 @@
#ifndef T_IOT_901_CONVOYOR_DOLIBARRMODELS_H
#define T_IOT_901_CONVOYOR_DOLIBARRMODELS_H
namespace models {
Owner

je suis chaud que tu m'explique les namesapce j'ai jamais utiliser ;)

je suis chaud que tu m'explique les namesapce j'ai jamais utiliser ;)
Nicolas marked this conversation as resolved
platformio.ini Outdated
@ -50,6 +50,9 @@ upload_speed = 921600
; librairies (make sure to fix versions where possible!)
lib_deps =
bblanchon/ArduinoJson@^6.21.3
Owner

mettre un commentaire sur chaque ligne pour dire l'utilité de la lib

mettre un commentaire sur chaque ligne pour dire l'utilité de la lib
Nicolas marked this conversation as resolved
src/Program.cpp Outdated
@ -2,4 +4,2 @@
Program::Program() {
// Startup
Serial.begin(MONITOR_SPEED);
Owner

remettre le serial print comme c'était

remettre le serial print comme c'était
Nicolas marked this conversation as resolved
@ -2,4 +5,4 @@
Program* program;
void setup() {
Owner

le main.cpp doit téoriquement rester par defaut

le main.cpp doit téoriquement rester par defaut
Nicolas marked this conversation as resolved
Clement added 1 commit 2023-11-10 10:51:10 +00:00
Nicolas added 2 commits 2023-11-10 15:33:39 +00:00
Nicolas added 1 commit 2023-11-10 15:42:13 +00:00
Nicolas requested review from Clement 2023-11-10 15:42:32 +00:00
Mathis approved these changes 2023-11-10 15:44:20 +00:00
Clement approved these changes 2023-11-10 15:44:31 +00:00
Clement added 1 commit 2023-11-10 15:45:23 +00:00
Clement merged commit 88077f284e into develop 2023-11-10 15:47:41 +00:00
Clement deleted branch feat/client_dolibarr 2023-11-10 15:47:52 +00:00
Sign in to join this conversation.
No description provided.