search

DrMemCS / drmem

Full source tree for the DrMem control system
MIT License
3 stars 4 forks source link

Fixing Redis #23

Closed rneswold closed 1 year ago

rneswold commented 1 year ago

This pull request, when ready, will fix the redis backend so it supports the full GraphQL API.

rneswold commented 1 year ago

At this point, the redis backend only supports one GraphQL query: get device info.

rneswold commented 1 year ago

I did some more testing and there were problems with some device queries. This set of commits fixed the bug, among other things (minor code refactoring, formatting, etc.)

By the way, the redis backend is able to use the "globbing" pattern syntax, so it's ahead of the simple backend, in this regard. This means that #7 is partially complete.

rneswold commented 1 year ago

Adding the setDevice was easier than I thought. 😄 Since settings go straight to the driver, there's no interaction with the backend, so no redis commands were necessary. Makes me think that this API call doesn't belong in the backend. 🤔