Skip to content

Link to GH release page of current version#255

Merged
rwjblue merged 1 commit intoember-cli:masterfrom
pangratz:change-link-in-help-menu
Oct 28, 2015
Merged

Link to GH release page of current version#255
rwjblue merged 1 commit intoember-cli:masterfrom
pangratz:change-link-in-help-menu

Conversation

@pangratz
Copy link
Copy Markdown
Contributor

This changes the link in the help menu to point to the release notes of the current version. Example: https://github.com/ember-cli/ember-twiddle/releases/tag/0.4.14


Note that the link will be broken in canary and in development, but I think that is acceptable...

@Gaurav0
Copy link
Copy Markdown
Contributor

Gaurav0 commented Oct 28, 2015

I think it should at least be possible to provide an alternate link in canary and development.

@pangratz
Copy link
Copy Markdown
Contributor Author

It could be the link to the specific commit in canary, but I am unsure what this should be in development. Maybe remove the item when in development? Or use https://github.com/ember-cli/ember-twiddle?

@pangratz
Copy link
Copy Markdown
Contributor Author

I updated the {{user-menu}} component in the following way:


No version link is shown when in development:

screen shot 2015-10-28 at 15 34 53


Link to commit is shown when in staging (aka canary):

screen shot 2015-10-28 at 15 35 43


Link to release is shown when in production:

screen shot 2015-10-28 at 15 36 07

- no link is shown when in development
- link to latest commit when in staging (aka canary)
- link to release notes when in production
rwjblue added a commit that referenced this pull request Oct 28, 2015
Link to GH release page of current version
@rwjblue rwjblue merged commit 0beac98 into ember-cli:master Oct 28, 2015
@pangratz pangratz deleted the change-link-in-help-menu branch October 28, 2015 15:01
@joostdevries
Copy link
Copy Markdown
Member

😍

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.

4 participants