Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Framework: Display the name of the current branch with the environment badge #1531

Merged
merged 2 commits into from
Dec 15, 2015

Conversation

Tug
Copy link
Contributor

@Tug Tug commented Dec 12, 2015

The idea is that it is useful to be remembered of the current branch when you are testing.

This is what it looks like:

cloudup-08a1b31a-7023-4396-a76d-864ef226fa22

Branch name is not displayed when:

  • Git is not available
  • The project is not a git repository
  • The current branch is master

Also, this is going to be really useful for a multiple versions application of calypso.

…t badge

The idea is that it is useful to be remembered of the current branch when you are testing.

Branch name is not displayed when:
- Git is not available
- The project is not a git repository
- The current branch is master

Also, this is going to be really useful for a multiple version application of calypso.
@Tug Tug added Accessibility (a11y) [Feature] Tracks Metrics & Monitoring Capturing analytics about user behavior on WordPress.com. WordAds labels Dec 12, 2015
@lancewillett lancewillett added the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Dec 14, 2015
@Tug Tug added Framework and removed Accessibility (a11y) WordAds [Feature] Tracks Metrics & Monitoring Capturing analytics about user behavior on WordPress.com. labels Dec 14, 2015
@Tug
Copy link
Contributor Author

Tug commented Dec 14, 2015

cc @lancewillett, @mtias

@lancewillett
Copy link
Contributor

Quick notes from testing the branch in Firefox, local install of latest master: I switched to this branch, but the badge didn't update. :(

screen shot 2015-12-14 at 12 47 21

@lancewillett
Copy link
Contributor

OK — after a new make run it's showing up! Nice.

screen shot 2015-12-14 at 12 50 10

@Tug Tug force-pushed the add/dev-show-branch-name branch from 9006204 to 55f5a20 Compare December 14, 2015 20:17
@Tug
Copy link
Contributor Author

Tug commented Dec 14, 2015

Let's wait for #1322 to be merged and review this change again!

@mtias
Copy link
Member

mtias commented Dec 15, 2015

@Tug this seems like a nice improvement. I don't think we need to wait on the design update to get it in, though :)

@Tug
Copy link
Contributor Author

Tug commented Dec 15, 2015

Ok :) let's merge it then?

@lancewillett
Copy link
Contributor

🚢

@mtias
Copy link
Member

mtias commented Dec 15, 2015

👍

@mtias mtias removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Dec 15, 2015
Tug added a commit that referenced this pull request Dec 15, 2015
Framework: Display the name of the current branch with the environment badge
@Tug Tug merged commit f12e5cd into master Dec 15, 2015
@mtias mtias deleted the add/dev-show-branch-name branch December 15, 2015 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants