Skip to content
This repository was archived by the owner on Aug 28, 2025. It is now read-only.

feat : upsert sensor#232

Open
anshulchahar wants to merge 32 commits intodevfrom
feat/upsert_sensor
Open

feat : upsert sensor#232
anshulchahar wants to merge 32 commits intodevfrom
feat/upsert_sensor

Conversation

@anshulchahar
Copy link
Copy Markdown
Contributor

@anshulchahar anshulchahar commented Feb 20, 2025 •
edited by DaL31
Loading

  • Added a function to upsert the sensor.
  • Replaces the old Sensor information with the new if id is already found in the Database
  • Inserts a new sensor if id is not found in the Database
  • Editing of an existing sensor is currently not working and needs to be implemented

Needs: #231 and #235

anshulchahar and others added 17 commits February 17, 2025 16:48
@anshulchahar
added the insert function
3c16911
@anshulchahar
added action for inserting a sensor
d551544
@anshulchahar @DaL31
Update src/actions/action_createSensor.ts
6f2bbe4 
Co-authored-by: David Le <53024307+DaL31@users.noreply.github.com>
@anshulchahar
fix: fix comment formatting
b5ecc94
@anshulchahar @DaL31
Update src/components/mongodb/DBConnector.tsx
b18262e 
Co-authored-by: David Le <53024307+DaL31@users.noreply.github.com>
@anshulchahar
Merge branch 'dev' into feat/insert_sensor
b50db06
@anshulchahar @PatrickBaus
Update src/components/mongodb/DBConnector.tsx
c777e77 
Co-authored-by: Patrick Baus <patrick.baus@physik.tu-darmstadt.de>
@anshulchahar @PatrickBaus
Update src/components/mongodb/DBConnector.tsx
5714f05 
Co-authored-by: Patrick Baus <patrick.baus@physik.tu-darmstadt.de>
@anshulchahar
feat: replaced insert function with upsert
e1ef546
@anshulchahar
removed the insertSensor funciton
99fbb3b
@anshulchahar
feat: deleted the action_createSensor.ts file
f85c248
@anshulchahar
feat: added action_upsertSensor.ts file
da86631
@anshulchahar
fix: fixed the comment
572dd84
@anshulchahar
fix: fixed comment
212ed10
@anshulchahar @PatrickBaus
Update src/components/mongodb/DBConnector.tsx
99abd11 
Co-authored-by: Patrick Baus <patrick.baus@physik.tu-darmstadt.de>
@anshulchahar
fix: import UUID from same source in every file
aa80380
@anshulchahar
ensure date_created is an Date object
a3565df
@anshulchahar
Copy link
Copy Markdown
Contributor Author

anshulchahar commented Feb 20, 2025

Last 2 commits are new.
I have tested it, It works.

PatrickBaus
PatrickBaus suggested changes Feb 20, 2025
View reviewed changes
Comment thread src/actions/action_upsertSensor.ts Outdated
Comment thread src/components/mongodb/DBConnector.tsx Outdated
Comment thread src/models/zTinkerforgeSensor.schema.ts Outdated
@anshulchahar @PatrickBaus
Update src/actions/action_upsertSensor.ts
0a7c650 
Co-authored-by: Patrick Baus <patrick.baus@physik.tu-darmstadt.de>
@anshulchahar
Copy link
Copy Markdown
Contributor Author

anshulchahar commented Feb 20, 2025 •
edited
Loading

There is some Textencoder error while running Tinkerforge schema test.
It was working before without a Problem ?

DaL31
DaL31 suggested changes Feb 21, 2025
View reviewed changes
Comment thread src/components/mongodb/DBConnector.tsx
@anshulchahar anshulchahar marked this pull request as draft February 21, 2025 16:42
@anshulchahar anshulchahar marked this pull request as ready for review March 11, 2025 13:13
anshulchahar and others added 5 commits March 11, 2025 14:13
@anshulchahar
Merge branch 'dev' into feat/upsert_sensor
aee6e75
@GGalya1
Merge branch 'dev' into feat/upsert_sensor
ad7368f
@DaL31
Merge branch 'dev' into feat/upsert_sensor
a9fb372
@GGalya1
Merge branch 'dev' into feat/upsert_sensor
5af9a25
@GGalya1
feature: implemented upserSensor function in the UI
d1afdbd 
- added opportunity to add sensors in database
- a test for a component was removed because there was a problem with an environment variable that did not understand what it depended on (previously all tests passed).
@GGalya1 GGalya1 requested review from DaL31 and PatrickBaus March 20, 2025 15:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@PatrickBaus PatrickBaus Awaiting requested review from PatrickBaus

@DaL31 DaL31 Awaiting requested review from DaL31

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@anshulchahar @PatrickBaus @DaL31 @GGalya1