-
Notifications
You must be signed in to change notification settings - Fork 1
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
chore: remove SS and rfs from the code base #83
Conversation
} | ||
for (const sensorId in responseJson.sensationalSensors) { | ||
sensorsArray.push(responseJson.sensationalSensors[sensorId]); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
right this is referencing the old react code which is not in use! should we add more red here and delete the react /app
folder too? @mikevespi
cc: @BallardRobinett
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah I think it would make sense to just the whole /app folder if possible
@@ -24,7 +24,7 @@ sleep .5 | |||
echo "looking up docker containers..." | |||
docker container ls -a | |||
sleep .5 | |||
if [ "$( docker inspect --format '{{json .State.Running}}' real-fake-sensors )" = "true" ] && [ "$( docker inspect --format '{{json .State.Running}}' sensational-sensors )" = "true" ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch on this!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved from FE perspective with a small comment!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking good on my local
With real sensor data coming into the system we no longer need Real Fake Sensors and Sensational Sensors in the code base.