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.

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

[P1] Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning #99

Open
CESARDELATORRE opened this issue Feb 10, 2024 · 3 comments
Open

[P1] Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning #99

CESARDELATORRE opened this issue Feb 10, 2024 · 3 comments
Assignees
@3lcarry
Labels
control-plane-app duplicate This issue or pull request already exists issue Bug, but it's not breaking the app P1 (High priority) High priority means fixes/featureswe should be working in DEV branch asap.Also, docs if high prio. testing

Comments

@CESARDELATORRE
Copy link
Contributor

CESARDELATORRE commented Feb 10, 2024 •
edited
Loading

  • Add Ethernet type to Control plane app camera types
  • Also, instead of Wifi --> Wi-Fi
image

From "Process" to "Container"?

Don't change it to "Container" yet. Let's discuss it in case we want to have several "Models" of Process.
See this other ISSUE:

#100

image
@CESARDELATORRE CESARDELATORRE added bug Something isn't working P2 (Less priority) Features or improvements with lower priority to do in upcoming weeks. control-plane-app labels Feb 10, 2024
@CESARDELATORRE CESARDELATORRE changed the title Add "Ethernet type" to cameras in Control plane app when provisioning Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning Feb 11, 2024
@CESARDELATORRE CESARDELATORRE added issue Bug, but it's not breaking the app and removed bug Something isn't working labels Feb 11, 2024
@CESARDELATORRE CESARDELATORRE changed the title Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning [P2] Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning Feb 11, 2024
@CESARDELATORRE CESARDELATORRE added P1 (High priority) High priority means fixes/featureswe should be working in DEV branch asap.Also, docs if high prio. and removed P2 (Less priority) Features or improvements with lower priority to do in upcoming weeks. labels Feb 14, 2024
3lcarry pushed a commit that referenced this issue Feb 16, 2024
Camera details
13cafd7 
Added camera details modal #109 and new types #99
@3lcarry
Copy link

3lcarry commented Feb 16, 2024

I push a new version, we need to reset the mongodb. I didn't realize we were storing the enum id instead of the string. This will allow us more flexibility to add new fields if needed.

@CESARDELATORRE
Copy link
Contributor Author

Not closing it because it's still a bug, here, related to this:

#122

@CESARDELATORRE CESARDELATORRE added the duplicate This issue or pull request already exists label Feb 17, 2024
@CESARDELATORRE CESARDELATORRE mentioned this issue Feb 17, 2024
Closed
@CESARDELATORRE CESARDELATORRE changed the title [P2] Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning [P1] Add "Ethernet type" and change from "Process" --> "Container" in cameras in Control plane app when provisioning Feb 17, 2024
@CESARDELATORRE
Copy link
Contributor Author

Closing, fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@3lcarry 3lcarry

Labels
control-plane-app duplicate This issue or pull request already exists issue Bug, but it's not breaking the app P1 (High priority) High priority means fixes/featureswe should be working in DEV branch asap.Also, docs if high prio. testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@CESARDELATORRE @3lcarry