Skip to content

Conversation

@mojavelinux
Copy link
Contributor

This PR backports changes added to production UI and build since the cloud UI was forked from it. Except for a few minor style changes, this does not fundamentally change the appearance or behavior of the cloud UI. Once this is merged, the cloud UI will become the official new UI.

@mojavelinux mojavelinux requested a review from amarantha-k June 16, 2020 19:25
@amarantha-k
Copy link
Contributor

@mojavelinux PR#16 & #17 look good and are ready to be merged. Just confirming that I can merge them before this PR

@mojavelinux
Copy link
Contributor Author

mojavelinux commented Jun 17, 2020 via email

@mojavelinux
Copy link
Contributor Author

I'll rebase this PR.

- delegate to helper to compute latest page URL (latest-page-url)
- delegate to helper to compute canonical URL (canonical-url)
- update latest page URL resolution to consider aliases
- update canonical URL resolution to consider aliases
- don't show prerelease banner if all versions are prereleases
- document the logic for computing the latest page URL and canonical URL
- hide View Latest button if page is not available in latest version
@mojavelinux
Copy link
Contributor Author

Rebase complete.

Copy link
Contributor

@amarantha-k amarantha-k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@amarantha-k amarantha-k merged commit d9c0648 into couchbase:cloud Jun 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants