Feature #8934

Assistant: Summarize boot menu key options

Added by sajolida 2015-02-21 19:47:46 . Updated 2015-09-29 07:07:04 .

Status:
Resolved
Priority:
Elevated
Assignee:
Category:
Installation
Target version:
Start date:
2015-02-21
Due date:
% Done:

100%

Feature Branch:
Type of work:
Research
Starter:
Affected tool:
Installation Assistant
Deliverable for:

Description

Building on the results from Feature #8804, let’s try to see if we can summarize all that information.


Subtasks


Related issues

Blocks Tails - Feature #9203: Write content for all scenarios Resolved 2015-01-27

History

#1 Updated by BitingBird 2015-04-10 16:48:18

  • Subject changed from Summarize boot menu key options to Assistant: Summarize boot menu key options

#2 Updated by tchou 2015-04-30 13:52:49

  • Blueprint set to https://tails.boum.org/blueprint/bootstrapping/booting/

Link to the blueprint.

#3 Updated by sajolida 2015-05-19 19:23:34

  • Affected tool set to Installation Assistant

#4 Updated by sajolida 2015-08-15 08:26:13

  • Assignee changed from tchou to sajolida

I’ll take this over.

#5 Updated by sajolida 2015-09-06 05:10:32

  • Assignee changed from sajolida to tchou
  • QA Check set to Ready for QA

In the end the information that we had on all the web pages were basically copied versions of one another. Some of this information was outdated (NEC, eMachines, etc.). So I simplified it, checked some user guides of recent laptop for sanity check.

So please review, you can do git log -p origin/master...origin/web/8934-boot-menu-keys for example. Then I’ll merge it into master.

#6 Updated by sajolida 2015-09-24 08:34:36

  • Target version set to Tails_1.7

We need to have this ready for the testing session in November so marking for 1.7.

#7 Updated by sajolida 2015-09-24 11:20:57

#8 Updated by tchou 2015-09-25 00:49:23

  • Assignee changed from tchou to sajolida
  • QA Check changed from Ready for QA to Info Needed

I have a lot of logs with your command, so I could not review it. I think that anyway we will have to improve this issue in 2016. Maybe past it here ? (I’m ok not to review it and try to improve it later).

#9 Updated by sajolida 2015-09-25 09:33:26

  • Assignee changed from sajolida to tchou

Then try:

  • `git log -p 46ac735..f583a16`
  • `git diff origin/master…origin/web/8934-boot-menu-keys`

Pasting the result only would prevent you from reading my commit messages and understand the process that lead me to such result.

I’m not sure to understand your comment about 2016. Personally, I want to have this out of my way for 2016. I would be up for incremental improvements on this list but I don’t see what would require a complex rewrite (or at least I’m not volunteering to work on this).

#10 Updated by tchou 2015-09-28 02:03:17

  • Assignee changed from tchou to sajolida

sajolida wrote:
> Then try:
>
> * `git log -p 46ac735..f583a16`
> * `git diff origin/master…origin/web/8934-boot-menu-keys`

Il failed because of .po issues (I had no time to fix) but I could read it with the web interface.

> Pasting the result only would prevent you from reading my commit messages and understand the process that lead me to such result.

ok

> I’m not sure to understand your comment about 2016. Personally, I want to have this out of my way for 2016. I would be up for incremental improvements on this list but I don’t see what would require a complex rewrite (or at least I’m not volunteering to work on this).

Maybe the misunderstanding come from me : I was not only thinking about this list, but about the all booting issue. I feel it’s full of corner and specific cases (wich windows version, wich BIOS…).

#11 Updated by sajolida 2015-09-28 10:10:41

  • Assignee changed from sajolida to tchou
  • Priority changed from Normal to Elevated
  • QA Check changed from Info Needed to Ready for QA

Unfortunately, you’re not really reviewing the implementation work I did on this branch. Could you please confirm that the list I prepared looks good enough for you to get included in the Installation Assitant as to be release in December?

#12 Updated by tchou 2015-09-28 13:18:18

  • Assignee changed from tchou to sajolida
  • QA Check deleted (Ready for QA)

Sorry, I let it implicit in “I could read it with the web interface”.
Yes it’s ok.

#13 Updated by sajolida 2015-09-29 04:45:07

  • Assignee deleted (sajolida)

Well, this received this time. Indeed removing “Ready for QA” or setting it as “Pass” also works.

So I merged this into master. This was the last feature branch on the Windows scenario. So it’s not fully finalized, reviewed, and all. Yoohoo!

I’ll proceed with moving this document into web/assistant and removing it from the blueprint. I won’t submit a branch for doing this as I won’t modify the content.

#14 Updated by sajolida 2015-09-29 04:45:40

  • Status changed from Confirmed to Resolved
  • % Done changed from 0 to 100

Applied in changeset commit:3d578d608d88f31afadafae7c0c5130387993461.

#15 Updated by sajolida 2015-09-29 07:07:04

Done. See commits 472b789 and 5df752a. I did them directly in web/assistant because they implied no change to the content. Still, feel free to review and build them and tell me if I fucked up anything.