Skip to content

Conversation

@jeahyoun
Copy link
Contributor

@jeahyoun jeahyoun commented Jun 12, 2018

Summary

This PR enables the feature flag to use the new UHF. See the work item here (https://ceapex.visualstudio.com/Engineering/UX%20Team/_backlogs/board/Stories) and the related feature ticket here (https://ceapex.visualstudio.com/Engineering/UX%20Team/_workitems/edit/18194).

This change can be merged into master for testing on our review site. The feature is not live yet so this merge would have no effect on production presentation.

Internal review site

edit by @BillWagner: Add internal review link

This PR enables the feature flag to use the new UHF. See the work item here (https://ceapex.visualstudio.com/Engineering/UX%20Team/_backlogs/board/Stories) and the related feature ticket here (https://ceapex.visualstudio.com/Engineering/UX%20Team/_workitems/edit/18194).

This change can be merged into master for testing on our review site. The feature is not live yet so this merge would have no effect on production presentation.
@BillWagner BillWagner changed the title Add "enable_uhf_ppe" feature flag to docfx.json. WIP Add "enable_uhf_ppe" feature flag to docfx.json. Jun 12, 2018
@BillWagner
Copy link
Member

Add "WIP" to title.

@mairaw
Copy link
Contributor

mairaw commented Jun 12, 2018

It seems this should be merged @BillWagner.

@BillWagner
Copy link
Member

@mairaw I've got an offline question to @jeahyoun. Waiting to hear back.

@mairaw mairaw added the blocked Blocked for some reason label Jun 14, 2018
@jeahyoun
Copy link
Contributor Author

Hey @BillWagner, I see a note that you have an offline question, but I didn't see any notification of this. I'm not sure if something slipped through the cracks (I didn't see anything in my email, in Teams, or in the VSTS ticket). I hope I didn't miss anything from you. If you resend, I can address this. Thanks!

@BillWagner
Copy link
Member

@jeahyoun I just resent the email.

@adkinn
Copy link
Contributor

adkinn commented Jun 27, 2018

@BillWagner @mairaw @jeahyoun any open questions here? We've shipped our code and we'd like to start the test.

@BillWagner
Copy link
Member

I don't have any concerns. When ready to merge, just remove the "WIP" label. We'll merge and see how that looks on the master branch on review. If there are no issues, we'll push live.

@adkinn adkinn changed the title WIP Add "enable_uhf_ppe" feature flag to docfx.json. Add "enable_uhf_ppe" feature flag to docfx.json. Jun 27, 2018
@adkinn
Copy link
Contributor

adkinn commented Jun 27, 2018

@BillWagner done and thanks!

Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks for confirming all the testing work.

I'll :shipit: now @jeahyoun @adkinn

@BillWagner BillWagner merged commit eb6fa20 into dotnet:master Jun 27, 2018
@BillWagner BillWagner removed the blocked Blocked for some reason label Jun 27, 2018
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