@Shubhamverma2796 Hasn't this been solved already for 4.2 with your merged PR #36860 ???

It is a different page #36860 (comment). PR #36860 fixed the manual upload package page.

And why is the other one for 4.2-dev and this one for 4.1-dev?

And why is it in draft status?

@Shubhamverma2796 Is it by purpose that this PR is still in draft status? Or is that by mistake? If the latter, you can change it by using the "Ready for review" button at the bottom of your PR. I think people will not spend time for testing as long as it is in draft status.

@Shubhamverma2796 Is it by purpose that this PR is still in draft status? Or is that by mistake? If the latter, you can change it by using the "Ready for review" button at the bottom of your PR. I think people will not spend time for testing as long as it is in draft status.

Actually I opened the pr but later I realised that it is not the right fix so I changed it to draft. I am working on it.

Ah, ok, thanks for the feedback.

Thank you for this PR but could we just remove the checkbox as its useless in my opinion. It just creates a barrier and it will not really educate people to backup.

It just creates a barrier

As intended so that they stop and think

and it will not really educate people to backup.

Based on ?

Based on comments here: #35583 and here: #36151 (you even liked the suggestion to remove the checkbox back then)

Based on comments here: #35583 and here: #36151 (you even liked the suggestion to remove the checkbox back then)

Please test PR #36531.

Thank you for this PR but could we just remove the checkbox as its useless in my opinion. It just creates a barrier and it will not really educate people to backup.

so should I remove the checkbox?

Thank you for this PR but could we just remove the checkbox as its useless in my opinion. It just creates a barrier and it will not really educate people to backup.

so should I remove the checkbox?

@Shubhamverma2796 As long as it is just an opinion in a discussion and not supported by a huge number of people, there is no need to remove the check box.

@coolcat-creations Would Nik's PR #36531 be a way? Beside other changes it makes the presence of the check boxes configurable.

@richard67 I have to check - it's a long read , thank you for directing me to this PR :)

@Shubhamverma2796 This probably should be rebased for v4.2 as @richard67 stated. Also why not use disabled attribute like you did with #36860?

Not using disabled attribute because :

1. There is no disabled attribute for anchor tag like this(a href="#" disabled") . Therefore, in order to use the disabled attribute, I will need to change the anchor tag to the button tag, which will cause problems for other empty states. For example: "Add your first article " will stop working.
2. Even if I do use the button tag and add the disabled attribute, then initially, I need to set disabled=true because initially, the checkbox will be unchecked so that the button must be disabled. Again, this will cause problems for other states, ex: the "Add your first article" button will become disabled making it inaccessible.

I first used the disabled attribute but later faced these problems therefore added the disabled to the class only. I took all the steps as per my understanding, please correct me if I'm wrong.

I have tested this item ✅ successfully on fbf3353

I have tested this item ✅ successfully on fbf3353

RTC

Thanks everybody