Bug 539630 – Incomplete/broken link in dev manual

• Status: RESOLVED FIXED
• Product: Documentation

[Description Fernando (likewhoa) 2015-02-10 15:30:02 UTC]

The current devmanual has a broken link which points to the search query's advanced tab but it currently just goes to the standard search tab. This fix includes the proper linkage and also adjustments to the text describing the link.

Reproducible: Always

Steps to Reproduce:
1.click the link near "for lazy people"
2. see no custom settings



Expected Results:  
working linkage

[Comment 1 Fernando (likewhoa) 2015-02-10 15:31:52 UTC]

Created attachment 396092 [details, diff]
devmanual broken link fix

[Comment 2 Ulrich Müller 2015-02-10 16:08:56 UTC]

(In reply to Fernando (likewhoa) from comment #1)
> Created attachment 396092 [details, diff] [details, diff]
> devmanual broken link fix

+select UNCONFIRMED and RESOLVED (RESOLVED is important), then

Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED?

[Comment 3 Fernando (likewhoa) 2015-02-10 16:10:05 UTC]

(In reply to Ulrich Müller from comment #2)
> (In reply to Fernando (likewhoa) from comment #1)
> > Created attachment 396092 [details, diff] [details, diff] [details, diff]
> > devmanual broken link fix
> 
> +select UNCONFIRMED and RESOLVED (RESOLVED is important), then
> 
> Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED?

I asked myself the same question but since it wasn't originally mentioned i left them out. Feel free to add as you see fit.

[Comment 4 Markos Chandras (RETIRED) 2015-02-10 18:22:31 UTC]

(In reply to Fernando (likewhoa) from comment #3)
> (In reply to Ulrich Müller from comment #2)
> > (In reply to Fernando (likewhoa) from comment #1)
> > > Created attachment 396092 [details, diff] [details, diff] [details, diff] [details, diff]
> > > devmanual broken link fix
> > 
> > +select UNCONFIRMED and RESOLVED (RESOLVED is important), then
> > 
> > Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED?
> 
> I asked myself the same question but since it wasn't originally mentioned i
> left them out. Feel free to add as you see fit.

Yes I think we need to mention all possible fields. Please attach an updated patch and I will commit it for you.

[Comment 5 Fernando (likewhoa) 2015-02-10 19:00:33 UTC]

Created attachment 396126 [details, diff]
updated patch that replaces previous one

[Comment 6 Fernando (likewhoa) 2015-02-10 19:01:10 UTC]

(In reply to Markos Chandras from comment #4)
> (In reply to Fernando (likewhoa) from comment #3)
> > (In reply to Ulrich Müller from comment #2)
> > > (In reply to Fernando (likewhoa) from comment #1)
> > > > Created attachment 396092 [details, diff] [details, diff] [details, diff] [details, diff] [details, diff]
> > > > devmanual broken link fix
> > > 
> > > +select UNCONFIRMED and RESOLVED (RESOLVED is important), then
> > > 
> > > Why only those, but not CONFIRMED, IN_PROGRESS, and VERIFIED?
> > 
> > I asked myself the same question but since it wasn't originally mentioned i
> > left them out. Feel free to add as you see fit.
> 
> Yes I think we need to mention all possible fields. Please attach an updated
> patch and I will commit it for you.

Updated and also removed non-existing fields.

[Comment 7 Fernando (likewhoa) 2015-02-10 19:03:47 UTC]

Created attachment 396128 [details, diff]
devmanual-fix-broken-link-539630-1.diff

This patch might be better since it suggest to select all possible fields in the status options list.

[Comment 8 Markos Chandras (RETIRED) 2015-02-15 09:57:30 UTC]

That looks ok. So now please attach a proper git patch and I will commit it for you :)

[Comment 9 Göktürk Yüksek 2016-05-01 03:56:04 UTC]

Merged. Thanks!

https://gitweb.gentoo.org/proj/devmanual.git/commit/?id=6e91b50253398d874fed5addbc08a2d8852d4fe3