feat: launch everything from the same docker compose #2
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
the name it had before made esphome think it was an esp32 config file
login_login is a pleonasme, it's useless to say login twice
it's the only required file for the database, therefore it doesn't need to be in another directory
jpilet
reviewed
Apr 10, 2024
.gitignore
Outdated
| .idea | ||
| # vim backup files | ||
| *.swp |
There was a problem hiding this comment.
I didn't knew you could do that, thank's
README.md
Outdated
| - esp32/config/secrets.example.yaml | ||
|
|
||
| --- | ||
| Once done you want to get the ip address of the computer that will run the docker containers |
There was a problem hiding this comment.
Is IP address mandatory or can we use a host name?
There was a problem hiding this comment.
I've changed it and it now use a host name
|  | ||
|
|
||
| ## ESP 32 Installation | ||
| This project uses esp home to manage the esp32, you can configure the esp32 by following the instructions below |
There was a problem hiding this comment.
Can you add a link to the esp home project?
README.md
Outdated
|
|
||
| --- | ||
| Finally you can fill the `esp32/config/secrets.yaml` with the credentials of the wifi\ | ||
| Once that's done you can create a postgres database called `memoires-info` from the `psql.sql` file |
There was a problem hiding this comment.
Db creation should be done with the POSTGRES_DB en Var. See doc https://hub.docker.com/_/postgres
See also the "Initialization scripts" section to pick psql.sql on the first execution.
before if you were making a request to anything.domain.com, the domain would be domain.com. Now the domain will be anything.domain.com it means that now everything starting with login.memoires-info.com will be run as php code instead of being ignored the reason it worked before was because the uri contained "login."
User now don't need to specify the port when accessing the web server
now postgres data won't apear in root
* feat: display averages for the last two months, refactor the way data is retrieved and placed in the pie chart * feat: token added to link for API data, change to display of month element * feat: link login element with login backend * feat: link login element with login backend * feat: the token in sampleContext takes the value from the user who logged in * feat: store the token in localstorage * fix: fix small errors to launch npm run build * feat: added login functionality, created a function to get tokens from localstorage * style: change input form design * feat: redirect the user on login if the token is wrong * feat: create a function that checks for 401 errors * refactor: the method of collecting data and putting it into the monthly chart * feat: add log out button
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.