search

autonomys / substation

Polkadot Telemetry service
GNU General Public License v3.0
3 stars 2 forks source link

Fix top bar updates #35

Closed i1i1 closed 2 years ago

i1i1 commented 2 years ago

This pr fixes stale of the top bar in the ui: image

As we don't send node updates to the frontend reactively, we need to send updates periodically. In future, we can just send all updates like that.

nazar-pc commented 2 years ago

Though I do think something like --update-interval CLI parameter would make more sense and you also need to update docker-compose.yml to include it.

i1i1 commented 2 years ago

Though I do think something like --update-interval CLI parameter would make more sense and you also need to update docker-compose.yml to include it.

Hmm, should we just have as close configuration in docker-compose.yml as possible to our production server then?

nazar-pc commented 2 years ago

Hmm, should we just have as close configuration in docker-compose.yml as possible to our production server then?

In subspace-customizations it is literally the same as in production, that was the point: you pull new changes, build and docker compose up -d. Do we run something different from what in the repo now?

i1i1 commented 2 years ago

Yeah, we don't disable node list on the frontend by default on production, so we apply changes locally on prod machine.

nazar-pc commented 2 years ago

I didn't understand what you said there, sounded like two conflicting statements