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

Add compile target to adopter registration stats #942

Merged
merged 1 commit into from
Sep 25, 2023

Conversation

LukasKalbertodt
Copy link
Member

@LukasKalbertodt LukasKalbertodt commented Sep 20, 2023

Since we already show it on the "about" page, including it in this data seems like a good idea. Currently this could be x86_64-unknown-linux-gnu or x86_64-unknown-linux-musl. Well, in theory it could be anything, who knows, maybe people manage to build Tobira for mipsel-sony-psp or armv6k-nintendo-3ds.

CC @gregorydlogan (though I'm fairly certain nothing else needs to change, as data is just piped through as is, if I remember correctly)

Since we already show it on the "about" page, including it in this
data seems like a good idea.
@github-actions github-actions bot temporarily deployed to test-deployment-pr942 September 20, 2023 19:18 Destroyed
@gregorydlogan
Copy link
Contributor

Opencast won't, but this does!

Also, I kinda want to see a 3DS running Tobira now...

@LukasKalbertodt
Copy link
Member Author

Ah oh. Can we still merge this? How would that thing then handle different versions of Tobira that send different amount of data?

@gregorydlogan
Copy link
Contributor

Yep, since the stats aren't being collected right now anyway :D

In the longer term, we add the additional fields to the registration server models. Fields with no data pushed end up full or blank string.

@owi92 owi92 merged commit 1a7a64b into elan-ev:master Sep 25, 2023
3 checks passed
@LukasKalbertodt LukasKalbertodt deleted the adopters-reg-data-target branch September 25, 2023 09:39
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