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

            links to

            Web Link Dev sandbox

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              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.
              Hide
              Permalink
              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
              Permalink
              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
              Permalink
              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
              Permalink
              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)

                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