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

battery #3115

Open
wants to merge 1 commit into
base: staging
Choose a base branch
from
Open

battery #3115

wants to merge 1 commit into from

Conversation

marvinbanda
Copy link

battery analysis tool

issue #629

battery analysis tool
Copy link
Contributor

@Baalmart Baalmart left a comment

Choose a reason for hiding this comment

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

Thanks @marvinbanda for this PR.

A few notes for your consideration...

  1. Our system is split into backend and frontend -- which makes it a 2 tier architecture, this particular repository is for our backend, so we do not expect frontend code here.
  2. You need to divide your solution into backend (API) and frontend accordingly. This approach ensures re-usability of your solution in many other areas.

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.

2 participants