Skip to content

Migrate deleting a boxes measurements #658

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.

Sign up for GitHub

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

Jump to bottom
Merged
scheidtdav merged 72 commits into from
Jan 20, 2026
Merged

Migrate deleting a boxes measurements #658

scheidtdav merged 72 commits into from
Jan 20, 2026

Conversation

@scheidtdav
Copy link
Collaborator

@scheidtdav scheidtdav commented Nov 19, 2025

Type of Change

  • Dependency upgrade
  • Bug fix (non-breaking change)
  • Breaking change
    • e.g. a fixed bug or new feature that may break something else
  • New feature
  • Code quality improvements
    • e.g. refactoring, documentation, tests, tooling, ...

Implementation

Implementation of existing endpoint

Checklist

  • I gave this pull request a meaningful title
  • My pull request is targeting the dev branch
  • I have added documentation to my code
  • I have deleted code that I have commented out

Additional Information

scheidtdav and others added 30 commits May 14, 2025 16:26
@scheidtdav
feat: add draft for port of user registration to resource route
35eeb87
@scheidtdav
feat: partly implement refresh token
42593f0
@scheidtdav
docs: simplify contributing and add info about api routes and shared …
dcd635f 
…logic
@scheidtdav
feat(api): finalize user registration endpoint
be4ebed
@scheidtdav
fix(tests): get the tests to run be reconfiguring build steps
b4b8421
@scheidtdav
docs(db): readd db setup and seed scripts with README info for it
8fbb075
@scheidtdav
fix: wrong import of utils
1de6e69
@scheidtdav
refactor: remove leftover custom server stuff
08f4405
@scheidtdav
fix(tests): add missing refresh token table
4a3f8e4
@scheidtdav
fix(tests): reenable remaining tests for registration
29d3034
@scheidtdav
fix(ci): remove playwright and use correct node version
d164738
@scheidtdav
fix(ci): run the tests with a postgres container
7566724
@scheidtdav
feat(tests): add coverage report
44894b4
@scheidtdav
fix(build): reorganize server modules to correctly split client/ server
77b4cc9
@scheidtdav
fix(build): miss an import
5612a5f
@scheidtdav
fix(build): remove leftovers from custom server implementation
2e335d5
@scheidtdav
chore(deps): bump react-router dependencies
bbf5430
@scheidtdav
chore(deps): update react-router
bf19c7e
@scheidtdav
feat/user me api (#559)
7d91045 
* feat(api): add api routes for /users/me

* fix(tests): api me PUT

* feat(api): add delete me endpoint
@scheidtdav
feat(api): add root route (#560)
a5699de
@JerryVincent
start
760914b
@JerryVincent
new commit
81a1f9c
@JerryVincent
tested docs
acf1770
@JerryVincent
added a route
eedb806
@JerryVincent
Added API Docs
a6245ea
@JerryVincent
modified
4712c6a
@JerryVincent
removed unsupported packages
f63dc07
@JerryVincent
updated
7741945
@JerryVincent
Modified
3b620a0
@JerryVincent
script generation without using ts-node.
f51f518
@scheidtdav scheidtdav linked an issue Nov 19, 2025 that may be closed by this pull request
migrate /boxes/:boxId/:sensorId/measurements #600
Closed
@scheidtdav scheidtdav marked this pull request as ready for review January 13, 2026 10:12
jona159
jona159 reviewed Jan 19, 2026
View reviewed changes
@scheidtdav scheidtdav enabled auto-merge (squash) January 20, 2026 12:57
@scheidtdav scheidtdav merged commit 31521da into dev Jan 20, 2026
9 of 10 checks passed
@scheidtdav scheidtdav deleted the feat/api-delete-sensor-data branch January 20, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jona159 jona159 jona159 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

migrate /boxes/:boxId/:sensorId/measurements

4 participants

@scheidtdav @jona159 @JerryVincent