BTW, new admin navbar is great! Big thanks to Heith and Dave for making it happen!
On Mon, Jan 11, 2016 at 4:54 PM, Igor Bondarenko <jetmi...@gmail.com> wrote: > I'm getting an error running `npm run build` inside docker on a fresh > master (3424bee). > > $ dc run web npm run build > > > allura@0.0.0 build /allura > > rm -rf Allura/allura/public/nf/js/build/; broccoli build > Allura/allura/public/nf/js/build/ > > File: context-menu.es6.js > SyntaxError: context-menu.es6.js: Unexpected token (27:21) > 25 | } > 26 | > > 27 | static propTypes = { > | ^ > > > Anyone knows why is that? > > Versions of nodejs and npm I have installed: > > $ dc run web bash -c 'node -v && npm -v' > v4.2.2 > 2.14.7 > > > On Sat, Jan 9, 2016 at 1:00 AM, Dave Brondsema <d...@brondsema.net> wrote: > >> For folks doing current development, the master branch now requires some >> `npm` >> commands to be run. The admin navbar code merged from >> https://forge-allura.apache.org/p/allura/tickets/7919/ starts using JS >> files >> that are written in ES6 and JSX syntax, so we have commands to transpile >> them >> into regular JS. You'll need to run these for the admin nav bar to work, >> which >> is now the way to add & edit tools. >> >> To install nodejs & npm, and run the commands, see the Installation >> instructions: >> >> https://forge-allura.apache.org/docs/getting_started/installation.html#a-few-more-steps-if-using-git >> >> You'll probably need to run that just once (unless you delete the >> transpiled.js >> file it creates). If you are editing the .es6.js files and want to >> transpile >> them frequently during development, see the npm commands in this section: >> >> https://forge-allura.apache.org/docs/development/contributing.html?highlight=broccoli#technology-stack >> >> -- >> Dave Brondsema : d...@brondsema.net >> http://www.brondsema.net : personal >> http://www.splike.com : programming >> <>< >> > >