Closed Bug 1445504 Opened 6 years ago Closed 6 years ago

"JavaScript Guide" tutorial does not include "Using Promises" page

Categories

(Developer Documentation Graveyard :: JavaScript, defect, P5)

All
Other
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mcguire, Assigned: cmills)

References

()

Details

:: Developer Documentation Request

      Request Type: Correction
     Gecko Version: unspecified
 Technical Contact: 

:: Details

The page https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Using_promises does not appear in the left-hand sidebar under "JavaScript Guide" (or anywhere else, for that matter). It is linked from the Promise object page (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Promise).
Hello,

Thanks for reporting this issue, the link indeed is missing... This sidebar is generated by a "macro" and its code lives on https://github.com/mdn/kumascript/blob/master/macros/JsSidebar.ejs

As you can see from L405 (https://github.com/mdn/kumascript/blob/master/macros/JsSidebar.ejs#L405 ) there is a bunch of links for the JavaScript Guide.

Do you feel confident to make a PR and fix this once and for all? :)
If not, that's no problem, I'll help.
Flags: needinfo?(mcguire)
Status: UNCONFIRMED → NEW
Ever confirmed: true
I've decided to jump in and fix this ;-)

I've submitted a PR to fix the sidebar:
https://github.com/mdn/kumascript/pull/658

I've added to promises guide to the main guide page:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide

I've added the nav menu to the promises guide itself:
https://developer.mozilla.org/en-US/docs/Web/JavaScript/Guide/Using_promises
Assignee: nobody → cmills
Flags: needinfo?(mcguire)
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
(In reply to rgucluer from comment #3)
> Hi,
> Problem continues:
> 
> "Using promises" link
> does not appear on some pages:

This is because the static pages had not been regenerated yet. I loaded the pages and did a force refresh (Shift + Refresh) on each page, and the "Using promises" link appears on all of them now.
Hi Chris,
  Thanks for the reply. Yes, all links are working now.
You need to log in before you can comment on or make changes to this bug.