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

[backend-frontend communication, frontend layout] #10

Merged
merged 3 commits into from
Nov 17, 2024

Conversation

kasprzakewa
Copy link
Collaborator

backend-frontend communication

  • endpoint for getting data from backend
  • date value added to esp_measurements for getting latest measurements
  • created WeatherDTO for efficient data transfer
  • added web configuration

frontend layout

  • implemented frontend layout
  • implemented http polling for cyclic data retrieval

**backend-frontend communication**
- endpoint for getting data from backend
- date value added to esp_measurements for getting latest measurements
- created WeatherDTO for efficient data transfer
- added web configuration

**frontend layout**
- implemented frontend layout
- implemented http polling for cyclic data retrieval
Copy link
Owner

@Bart3kTK Bart3kTK left a comment

Choose a reason for hiding this comment

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

Generalnie nie rozumiem dlaczego tworzyc nowa klase reprezetujaca pomiary

.vscode/launch.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
Copy link
Collaborator

@MaciejBinczarowski MaciejBinczarowski left a comment

Choose a reason for hiding this comment

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

Ogólnie to jest kilka drobnych rzeczy do zmiany moim zdaniem, ale musiałem się do czegoś przyczepić 😅 To poprawki na potem.
Ogólnie dobra robota! Dostarczym coś co działa i wygląda 😎

react/weather/src/App.tsx Show resolved Hide resolved
react/weather/src/components/History.tsx Show resolved Hide resolved
react/weather/src/components/Home.tsx Show resolved Hide resolved
react/weather/src/components/Home.tsx Show resolved Hide resolved
react/weather/src/components/Home.tsx Show resolved Hide resolved
@kasprzakewa kasprzakewa merged commit 8fe539f into main Nov 17, 2024
6 checks passed
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.

3 participants