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

Refactoring database structure #212

Merged
merged 88 commits into from
Oct 6, 2023
Merged

Conversation

loan21xw
Copy link
Collaborator

@loan21xw loan21xw commented Oct 6, 2023

Describe your changes

A lot of things

Issue ticket number and preferably a link

#127

This change belongs to the following squad(s)

  • Database
  • HTTP
  • OCPP

I have created unit tests for my code

  • Yes
  • No
  • No, I need help with this

Checklist before creating my pull request

  • My code is properly commented where needed
  • I have updated the documentation with my changes/additions
  • I have reviewed my code before creating this pull request
  • I have made sure that I am targeting the correct branch

sifudiep and others added 30 commits September 24, 2022 13:51
…se to use local instead of production database.
Created a function to search for the current tariff and if no tariffs are found some days worth will be generated and saved.
also updated respective DLL and interface.
and pasted the correct version of the dummy data in db.js.
AaronWheezer and others added 22 commits September 26, 2023 18:30
…lidation. Also cleaned up old comments in index.js
…s. Fixed some issues with the various test suites.
@AaronWheezer
Copy link
Collaborator

took me 4 days, thanks.... for wasting my time on this burned my eyes

@loan21xw
Copy link
Collaborator Author

loan21xw commented Oct 6, 2023

took me 4 days, thanks.... for wasting my time on this burned my eyes

You're welcome!

@loan21xw loan21xw merged commit 51bf2f7 into dev Oct 6, 2023
3 checks passed
@loan21xw loan21xw deleted the refactoring-database-structure branch October 6, 2023 16:02
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.

7 participants