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

Ble spam select device spam #2275

Open
theguythatdidsomething opened this issue Sep 29, 2024 · 11 comments
Open

Ble spam select device spam #2275

theguythatdidsomething opened this issue Sep 29, 2024 · 11 comments
Labels
enhancement New feature or request

Comments

@theguythatdidsomething
Copy link

Description of the feature you're suggesting.

Select what to spam like AirPods not random

Anything else?

No response

@theguythatdidsomething theguythatdidsomething added the enhancement New feature or request label Sep 29, 2024
@fr1sk
Copy link

fr1sk commented Oct 9, 2024

@theguythatdidsomething I can implement this, but let's first discuss how the user will choose what to send. We already have the random device button, so perhaps if iOS is selected, instead of the random device checkbox, we could display a left/right selector. The default option could be rnd device, but scrolling left or right would allow the user to select specific devices like AirPods, Apple TV, a new phone, etc. @jboone @eried @htotoo what do you think?

@theguythatdidsomething
Copy link
Author

theguythatdidsomething commented Oct 9, 2024

default

I love the idea it was just what i was thinking maybe also select the variable like green airpods max etc

@htotoo
Copy link
Member

htotoo commented Oct 9, 2024

@theguythatdidsomething I can implement this, but let's first discuss how the user will choose what to send. We already have the random device button, so perhaps if iOS is selected, instead of the random device checkbox, we could display a left/right selector. The default option could be rnd device, but scrolling left or right would allow the user to select specific devices like AirPods, Apple TV, a new phone, etc. @jboone @eried @htotoo what do you think?

Seems good, as long as it can fit the ext app filesize and memory usage. Feel free to implement that!
We'll see the details on the PR :)

@theguythatdidsomething
Copy link
Author

theguythatdidsomething commented Oct 9, 2024

@htotoo @fr1sk Last little question if this does get implemented how long would it take to be in a update
And would this make me a contributor

No pressure just curious

@gullradriel
Copy link
Member

If you implement it, it would make you a contributor, and in that case, the time it would take would depend on you.

Giving an idea and not doing the code => not making you a contributor.

If we are making it (@htotoo or anyone), then it will be ready when it will be ready.

Once ready it's generally quickly implemented in 'next' / available in the nightly following the merge.

And it would be in stable when we make a stable release. No dates for this, it's when we think we have enough new working things / fixes.

@htotoo
Copy link
Member

htotoo commented Oct 9, 2024

If it is urgent, you can use the BLE tx app. You just need to have a txt file with the ble packet in it. I think there are some available, so you can ty and copy.

@fr1sk
Copy link

fr1sk commented Oct 9, 2024

@theguythatdidsomething I should be able to deliver this in the next couple of days (most likely over the weekend). However, I can't guarantee it, as this is my first contribution to Mayhem, and I need some time to get familiar. I'm also balancing this with my regular work, so it's being done in my free time.

@theguythatdidsomething
Copy link
Author

If you implement it, it would make you a contributor, and in that case, the time it would take would depend on you.

Giving an idea and not doing the code => not making you a contributor.

If we are making it (@htotoo or anyone), then it will be ready when it will be ready.

Once ready it's generally quickly implemented in 'next' / available in the nightly following the merge.

And it would be in stable when we make a stable release. No dates for this, it's when we think we have enough new working things / fixes.

it sounds like i got you mad sorry if so i was just asking

@gullradriel
Copy link
Member

it sounds like i got you mad sorry if so i was just asking

Certainly not. You were asking, I answered.

Trust me, if I had gotten mad, you would clearly have noticed it, and it would not be an impression ;-)

@htotoo
Copy link
Member

htotoo commented Oct 9, 2024

@theguythatdidsomething I should be able to deliver this in the next couple of days (most likely over the weekend). However, I can't guarantee it, as this is my first contribution to Mayhem, and I need some time to get familiar. I'm also balancing this with my regular work, so it's being done in my free time.

Feel free to join discord and ask if you got any questions. But keep in mind, decs are from all around the world, so maybe have to wait for an answer . But anyway it worth to join :)

@theguythatdidsomething
Copy link
Author

theguythatdidsomething commented Oct 9, 2024

from
@htotoo

Thanks for the suggestion but i am already on there quite an active user in fact

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants