Added example (#25) - #28
Open
pragmayticer wants to merge 1 commit into
Open
Conversation
* Added example added plc server ran black optimized imports tryed to figure out what the AssesRetreiver does tryed to figure out what the AssesRetreiver does cleaned up code made more methods made some Comments made some Comments cleaned up Code cleaned up Code cleaned up Code cleaned up Code cleaned up Code introdiced DbmemoryAssetRetriever ran black and set other echo level ran black and set other echo level ran black and set other echo level added Tests and added that the AssetReriever now working from DB. ran black added test for PLC Server made complete test with plc server but got timestamp problems,tryed to fix them fixed Timestamp Error zwischenstand more tests ran black optimized imports zwischenstand added config for id in DB cleaned up code cleaned up code fixed json deserizable errors ran black tryed to fix errors added plc to requirements.txt added secret for plc-server-openapi-client old version commented in commented in fixed tests fixed tests removed print ran black made valid test Data removed prints * Added example added plc server ran black optimized imports tryed to figure out what the AssesRetreiver does tryed to figure out what the AssesRetreiver does cleaned up code made more methods made some Comments made some Comments cleaned up Code * cleaned up Code * Removed faker uuid Co-authored-by: julian <j.feinauer@pragmaticminds.de> added migrate removed old code
pragmayticer
force-pushed
the
24-add-database-for-assets
branch
from
June 9, 2022 06:52
da74883 to
c93bf4d
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
added plc server
ran black
optimized imports
tryed to figure out what the AssesRetreiver does
tryed to figure out what the AssesRetreiver does
cleaned up code
made more methods
made some Comments
made some Comments
cleaned up Code
cleaned up Code
cleaned up Code
cleaned up Code
cleaned up Code
introdiced DbmemoryAssetRetriever
ran black and set other echo level
ran black and set other echo level
ran black and set other echo level
added Tests and added that the AssetReriever now working from DB.
ran black
added test for PLC Server
made complete test with plc server but got timestamp problems,tryed to fix them
fixed Timestamp Error
zwischenstand
more tests
ran black
optimized imports
zwischenstand
added config for id in DB
cleaned up code
cleaned up code
fixed json deserizable errors
ran black
tryed to fix errors
added plc to requirements.txt
added secret for plc-server-openapi-client
old version
commented in
commented in
fixed tests
fixed tests
removed print
ran black
made valid test Data
removed prints
added plc server
ran black
optimized imports
tryed to figure out what the AssesRetreiver does
tryed to figure out what the AssesRetreiver does
cleaned up code
made more methods
made some Comments
made some Comments
cleaned up Code
cleaned up Code
Removed faker uuid
Co-authored-by: julian j.feinauer@pragmaticminds.de
made the main as main