PaaS improvements #375
Closed
PaaS improvements #375
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Stop Cloudfoundry duplicating dependency downloads.
User research has shown that designers are confused by this text. This is a suggested improvement that we can
User research shows this wasn’t hugely helpful in practice.
|
Thanks for this! That error does look clearer. When you say you'd like to test with users, do you mean you'd like this merged and released in order to do that? |
|
Yes please. Not all users have something ready to go, and so we ask them to
download and install the vanilla prototyping kit. It would be great if
we've sped things up for them and made the instructions clearer.
…On 19 April 2017 at 18:28, Joe Lanman ***@***.***> wrote:
Thanks for this! That error does look clearer.
When you say you'd like to test with users, do you mean you'd like this
merged and released in order to do that?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#375 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AOZBkyN4ItCINCnHSPdTqTREJz9MTt1wks5rxkQugaJpZM4NB-T8>
.
--
*Tom Dolan*
Senior Product Manager - Platform as a Service
<https://www.cloud.service.gov.uk/>
*Government as a Platform* | Government Digital Service | Cabinet Office
M: 07932 042614
W: https://gds.blog.gov.uk/
B: Aviation House | 125 Kingsway | London | WC2B 6NH
|
|
looks good, could you squash the last two commits together? They seem to iterate on the text, so I think we just need one. Also can you change the link URL to: https://govuk-prototype-kit.herokuapp.com/docs/publishing-on-heroku#6-set-a-username-and-password That's a bug that was there before your PR but we might as well fix it now |
charlesrt
pushed a commit
to UKHomeOffice/home_office_design_kit
that referenced
this pull request
Apr 24, 2017
# 5.1.1 - Update the alpha, beta and discovery colours to $govuk-blue to match the updated phase banner ([PR alphagov#370](alphagov/govuk_frontend_toolkit#370)) - Fix radio button show/hide behaviour when used outside a form ([PR alphagov#375](alphagov/govuk_frontend_toolkit#375)) - Fix a "Stick at top when scrolling" component bug related to scroll anchoring in Chrome 56+ ([PR alphagov#376](alphagov/govuk_frontend_toolkit#376)) - Minor travis fixes ([PR alphagov#373](alphagov/govuk_frontend_toolkit#373)) # 5.1.0 - Allow custom options when tracking events ([PR alphagov#365](alphagov/govuk_frontend_toolkit#365)) # 5.0.3 - Change HMRC and DEFRA text colours for improved contrast ([PR alphagov#368](alphagov/govuk_frontend_toolkit#368))
|
Cancelling this as getting the merge to work is proving too fiddly. :-) |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The PaaS team are conducting user research with the prototype kit.
We've found that the error message when basic auth hasn't been set isn't as helpful as it could be. We've not yet added a PaaS-specific error link, but hope to do that later once our own docs are ready for release. We would dearly like to test the new version of this error text with users in Newcastle next week.
We also found performance improvements by copying .gitignore to .cfignore - this prevented node.js dependencies being uploaded to PaaS by the user and then downloaded as part of the buildpack process on PaaS. Staging times were approximately halved for an unmodified prototyping kit as a result. This change should only affect Cloud Foundry users.