Comments (4)
To avoid this situation for v3 we should consider throwing an error when components are requested
at the latest version, or requested without a version range specified.
v3 already does this, it returns an error if no semver range/version is specified
origami-build-service/lib/middleware/v3/parseModulesParameter.js
Lines 79 to 81 in 7e1eadd
Note I don't think this should affect demo endpoints as demos are self contained — it would be reasonable
to embed a component demo for the latest version, the demo wouldn't break given a major release
The pull-request for v3 demos currently also returns an error if no semver range/version is specified. -- https://github.com/Financial-Times/origami-build-service/pull/460/files#diff-8c1eb431d6f3aeb9fdd100579f1a61e65164acf12417a903bb4a54b420541f56R52-R54
from origami-build-service.
v3 already does this
oh, nice!
The pull-request for v3 demos currently also returns an error
Cool, I think that's fine. I was thinking before that showing the latest demo could be useful for pages like the branding documentation, but using a specific version will always be safer (a future demo isn't guaranteed to show the same thing, in the same way). So let's leave it as is and close this issue until we have a more compelling reason to allow requests for the latest version of demos.
Turns out v2 errors when requesting the latest version of a demo since we restricted to valid semver ranges. It's broken the branding documentation latest
requests again,
since we don't know what other pages may be affected.
from origami-build-service.
Turns out v2 errors when requesting the latest version of a demo since we restricted to valid semver ranges. It's broken the branding documentation
🙈 I'll open a Build Service v2 PR to allowlatest
requests again,
That should work with the semver range *
rather than the latest
keyword, have you tried that?
from origami-build-service.
So let's leave it as is and close this issue until we have a more compelling reason to allow requests for the latest version of demos.
Closing this issue
from origami-build-service.
Related Issues (20)
- OBS v2: Allow the `latest` demos of a component to be returned
- Update to origami 2.0 components
- Update URL Updater HOT 1
- Limit v3 to bundling origami type "component" projects HOT 2
- Feature proposal: allow components to specify the dart-sass and ecmascript version they require to compile
- Request to improve the error message for build service v2 request containing new component HOT 1
- Improve surrogate keys used for V3 endpoints HOT 1
- Improve conflict messages
- Remove `.default` from component JS served via the v3 bundle endpoint HOT 3
- Investigate the caching issue on the url updater page
- system_code query parameter opt out HOT 1
- Bug: Base tag in HTML element is breaking links HOT 4
- Improve loading times of demos HOT 4
- bug: url-updater incorrectly reports a url as up-to-date if the requested modules have not defined a version
- Validate entire query param contents for unexpected characters
- Handle old Build Service V2 requests HOT 1
- Migrate from Origami bower to npm HOT 3
- Decommission Origami Build Service v2
- Migrate from Splunk Edge Forwarder
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from origami-build-service.