openandroidinstaller-dev / openandroidinstaller

Makes installing alternative Android distributions nice and easy.
https://openandroidinstaller.org
GNU General Public License v3.0
427 stars 33 forks source link

Add option for displaying specific text about a requirement #80

Open Blue-Sauce opened 1 year ago

Blue-Sauce commented 1 year ago

Some devices have requirements that if not met will result in a failed flash and brick. Could be a soft brick but still something you want to try hard to prevent. An example is the OnePlus Nord N200. Attempting to flash a custom rom while your device is on android 12 will result in a soft brick. I suggest adding the ability to display specific text about requirements and an optional field in the device config file to specify the message to display about the requirement. I'd also suggest a "brick risk" flag for cases where failing to meet the requirement is known for certain to result in a brick that is difficult or even impossible to recover from. In that case a large danger icon could be displayed to help communicate the danger to the user.

tsterbak commented 1 year ago

Hey @Blue-Sauce, first of all thank you for your interest in the project :) Your suggestion seems like a good idea!

There is currently the option to specify additional requirements in the config metadata (https://github.com/openandroidinstaller-dev/openandroidinstaller#how-to-write-metadata). Users will then be asked to check them off before they can continue. Currently the only custom options are Android version and Firmware version, but this could also be extended to fully custom requirements or the soft brink risk could be added as an additional check for users. What do you think?

Blue-Sauce commented 1 year ago

Before opening the issue I did take a look at how it looks, how the current additional requirements appear in OpenAndroidInstaller. Used my OnePlus Nord N200 (but did I did not complete the install, as I already have lineage on it). That's what prompted me make the suggestion. With the current wording I worry that some users might think it's ok to just try it, or think they probably meet the requirements. Not understand the consequences and the need to be absolutely sure.

On xda forums and on telegram several OnePlus Nord N200 users have missed the warnings in the xda guides and got bricked.

This is what I currently see for the android 11 requirement

Screenshot ![Screenshot](https://raw.githubusercontent.com/Blue-Sauce/misc/main/Screenshot-OAI-step2.png)
tsterbak commented 1 year ago

I see :thinking: How could the wording help people better?

anon1892 commented 1 year ago

Could be fixed with #221

tsterbak commented 8 months ago

@Blue-Sauce would you consider this resolved?

Blue-Sauce commented 8 months ago

Looking at the screenshot here https://github.com/openandroidinstaller-dev/openandroidinstaller/issues/114#issuecomment-1661718598 it seems good.

Though, I would feel better about it if an optional checkbox could be created for devices where it's known that a brick will occur if the user fails to heed the warning. So you could specify a checkbox with text such as this, "I understand that if my device is on android 12, flashing will result in a brick / unusable device."

anon1892 commented 8 months ago

I don't think it's very useful : is user does not read, he will neither read what's written next to the checkbox and will check it anyway...

Blue-Sauce commented 8 months ago

When one starts automating functions and making things point & click easy, as OAI aims to do, users start to think that it should be smart enough to detect issues that would destroy a device. And they can click without thinking properly. But of course, OAI is unable to detect fatal issues. So I feel it's reasonable to try to prevent damage as much as possible. I think checkbox can help a lot with that. It pulls attention to the warning. As it is, there is only the large "Pick OS Image" button to pull one's attention.