archive/rsnext
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions
rsnext/packages/next-server/amp.d.ts

2 lines
40 B
TypeScript
Raw Normal View History

Remove `withAmp` and do not expose `isInAmpMode` (#7690) * Remove `withAmp` and do not expose `isInAmpMode` This pull request removes the `withAmp` higher-order component in favor of the exported configuration object. Retaining this backwards compatibility isn't worth the extra maintenance burden, given it hasn't been released for very long and the migration is ultra-simple. I'm :-1: on retaining backwards compatibility just for the sake of it. Migration example follows. I'm willing to write a codemod if reviewers feel it's necessary. *Before* ```js import { withAmp } from 'next/amp' function Home() { return <h1>My AMP Page</h1> } export default withAmp(Home) // or export default withAmp(Home, { hybrid: true }) ``` *After* ```js export default function Home() { return <h1>My AMP Page</h1> } export const config = { amp: true, // or amp: 'hybrid', } ``` * Fix { amp: 'hybrid' } Co-Authored-By: JJ Kasper <jj@jjsweb.site>
2019-06-29 00:04:24 +02:00
export { useAmp } from './dist/lib/amp'
Reference in a new issue Copy permalink