Skip to content

Conversation

@danikasvetelj
Copy link

Changes made:

  • Added ping buttons and status messages to GUI for the following boards: fcb, pbb, daq, fsb, bms, cib, lrb.
  • Added an updated layout including a 'board status' section of the GUI. Note that this uses the image 'reload-arrows'.
  • Updated dependencies to get rid of dependency errors.

To test:

  • run npm install and npm run dev. You should be able to see the updated GUI.

Copy link

@connellr023 connellr023 left a comment

Choose a reason for hiding this comment

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

Looks good to me. I believe we already went through this on the old repository


export const usePocketbase = (timestamps: Timestamps, stores: Stores) => {
const pocketbase = new PocketBase('http://192.168.0.69:8090');
const pocketbase = new PocketBase('http://localhost:5173/:8090');

Choose a reason for hiding this comment

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

This looks like a mistake. Change to http://localhost:8090

# Initialize PocketBase
pb = Client("http://127.0.0.1:8090")
admin_email = "danikasvetelj@ucalgary.ca"
admin_password = "Ilmnh&h!Ilmnh&h!"

Choose a reason for hiding this comment

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

I hope you don't actually use this password 😆


export const usePocketbase = (timestamps: Timestamps, stores: Stores) => {
const pocketbase = new PocketBase('http://192.168.0.69:8090');
const pocketbase = new PocketBase('http://localhost:5173/:8090');

Choose a reason for hiding this comment

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

This needs to stay as the PI IP

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.

4 participants