Skip to content
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

SM-5 - persist protocol #212

Merged
merged 2 commits into from
Oct 11, 2023
Merged

SM-5 - persist protocol #212

merged 2 commits into from
Oct 11, 2023

Conversation

rhvivancoeffio
Copy link
Contributor

Proposed Changes

Impacted Routes

Caveats

@linear
Copy link

linear bot commented Oct 4, 2023

SM-5 devices-api device fingerprint topic on consume persist protocol

Need to persist the protocol from edge-network so we can then trigger changing the template

This consumer could also call code that updates the template.

Code where fingerprint is consumed:

func RunConsumer(ctx context.Context, settings *config.Settings, logger *zerolog.Logger, i *issuer.Issuer, dbs db.Store) error {

Once we get the protocol from here, we want to persist it to the user_devices.metadata column - their should already be an object here where you can set it. Do not update the table if the protocol is already set.

@JamesReate JamesReate merged commit 13e4173 into main Oct 11, 2023
2 checks passed
@JamesReate JamesReate deleted the devices-api-persist-protocol branch October 11, 2023 15:07
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.

3 participants