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

PaaS improvements to prototype kit #377

Closed
wants to merge 2 commits into from

Conversation

tomskerous
Copy link

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.

This change also addresses a mis-entered URL in the error message where
a user hasn’t set up authentication around their prototype.

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.

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.

This change also addresses a mis-entered URL in the error message where
a user hasn’t set up authentication around their prototype.

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.
@colmjude
Copy link

@tomskerous as discussed I've created a separate PR for this (see #380) so this can be closed now.

@joelanman joelanman closed this Apr 26, 2017
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.

None yet

3 participants