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 'Community creation' warning #1244

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

nekromant
Copy link
Contributor

I have made a few mini bles for me and friends based on the available specs. As @limpkin suggested I used serial numbers >= 100000 for them. To avoid any frustration (in case any of the devices end up on ebay), I suggest treating any devices with serial number >=100000 as 'community creations'. For them:

  • Security challenge is disabled to avoid any irrelevant spam to the (unexpecting) original team,
  • A warning is printed that the device is not affiliated with the original team.

The translation files are updated yet again, I've added relevant Russian translation for the new messages and (hopefully) not too ugly German one. (The latter one needs some checking by some native speaker though, my knowledge of German is pretty bad).

…h serial >=100000

Signed-off-by: Andrew 'Necromant' Andrianov <[email protected]>
Signed-off-by: Andrew 'Necromant' Andrianov <[email protected]>
@@ -7,7 +7,7 @@
<x>0</x>
<y>0</y>
<width>946</width>
<height>632</height>
<height>1081</height>
Copy link
Collaborator

Choose a reason for hiding this comment

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

i think that's a problem.... @deXol ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't remember explicitly changing these sizes, must be qt creator's doing. Will double-check tomorrow.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

<width>770</width>
<height>882</height>
<width>932</width>
<height>1832</height>
Copy link
Collaborator

Choose a reason for hiding this comment

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

tagging @deXol too

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

<width>646</width>
<height>885</height>
<width>524</width>
<height>704</height>
Copy link
Collaborator

Choose a reason for hiding this comment

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

here too...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

also fixed

@limpkin
Copy link
Collaborator

limpkin commented Sep 18, 2024

thanks a lot @nekromant !!! It's really appreciated.

@nekromant
Copy link
Contributor Author

Obligatory screenshot of how it looks like with a homebrew device:

Screenshot_20240918_233838

Signed-off-by: Andrew 'Necromant' Andrianov <[email protected]>
Signed-off-by: Andrew 'Necromant' Andrianov <[email protected]>
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