Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-1229

Restore API and Test API links to site

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.9.0
    • Component/s: site
    • Labels:
      None

      Description

      The site used to have links for API and Test API, but no longer has. It seems to have happened in the SVN change to fix CALCITE-1203.

      The other non-cosmetic change in that svn change, removing Michael Mior from the committers list, was my fault. I had committed to svn but forgot to push to git.

        Issue Links

          Activity

          Hide
          jcamachorodriguez Jesus Camacho Rodriguez added a comment -

          Resolved in release 1.9.0 (2016-09-22)

          Show
          jcamachorodriguez Jesus Camacho Rodriguez added a comment - Resolved in release 1.9.0 (2016-09-22)
          Hide
          elserj Josh Elser added a comment -

          Just realized I never looked at this one like I said I would. Looking at it after the fact, things look good. The whitespace fix for news articles will be nice (I never dug into it to figure out where all that was coming from).

          Show
          elserj Josh Elser added a comment - Just realized I never looked at this one like I said I would. Looking at it after the fact, things look good. The whitespace fix for news articles will be nice (I never dug into it to figure out where all that was coming from).
          Hide
          julianhyde Julian Hyde added a comment -

          Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/0c582435. Will be in release 1.9.0, and is already in the latest version of the site (published for the 1.8.0 release announcement).

          Show
          julianhyde Julian Hyde added a comment - Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/0c582435 . Will be in release 1.9.0, and is already in the latest version of the site (published for the 1.8.0 release announcement).
          Hide
          julianhyde Julian Hyde added a comment - - edited

          I figured it out. The new version of jekyll handles external_url differently, so the api and testapi pages were no longer matching in the for loop in docs_ul.html.

          Show
          julianhyde Julian Hyde added a comment - - edited I figured it out. The new version of jekyll handles external_url differently, so the api and testapi pages were no longer matching in the for loop in docs_ul.html .
          Hide
          julianhyde Julian Hyde added a comment -

          Specifically, see the changes to docs/index.html in r1739659, search for "/docs/api.html" and you see the API hyperlink has been removed.

          Show
          julianhyde Julian Hyde added a comment - Specifically, see the changes to docs/index.html in r1739659 , search for "/docs/api.html" and you see the API hyperlink has been removed.

            People

            • Assignee:
              julianhyde Julian Hyde
              Reporter:
              julianhyde Julian Hyde
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development