Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode - Mailing list pgadmin-hackers

From Dave Page
Subject Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode
Date
Msg-id CA+OCxoww3angrAPh+9wo4yq6un4h35Ht1ELzO-vRV9Kwnc5ZXg@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode  (Surinder Kumar <surinder.kumar@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode
List pgadmin-hackers


On Wed, Sep 13, 2017 at 10:34 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:
On Wed, Sep 13, 2017 at 2:52 PM, Dave Page <dpage@pgadmin.org> wrote:


On Wed, Sep 13, 2017 at 10:06 AM, Surinder Kumar <surinder.kumar@enterprisedb.com> wrote:

On Wed, Sep 13, 2017 at 2:18 PM, Dave Page <dpage@pgadmin.org> wrote:
Thanks, committed - though I did add a bundle-dev target to the Makefile for convenience.

I'm a little concerned though that the query history still doesn't work in dev mode. Is that fixable?
​That's an issue with ReactJS and I had logged a ticket​
 
​on their repository but didn't get any response from them.
The solution is pass `production` flag in place of `envType` in `dev` mode too.

const definePlugin = new webpack.DefinePlugin({
​  ​
'process.env': {
​     ​
'NODE_ENV': JSON.stringify(envType),
​ // In dev mode envType is `development`​

​  ​
},
});

What would be the effect of that? No source map being generated for React?
​Only React is using definePlugin's '
NODE_ENV'
​ variable​
for writing debug-only statements in React code.
​ It will have no effect on functionality. It is used to reduce React bundle size by eliminating debug-only code when run in `production` mode.

React code is like:

function implementSomeReactBehavior() {     // do actual work part 1      if(process.env.NODE_ENV !== "production") {         // do debug-only work, like recording perf stats     }      // do actual work part 2  
}

We are not generating source maps for vendor libraries including React. We generate source maps for pgAdmin4 modules only for debugging purpose.

OK, in that case can you please tweak our build so React is always in prod mode, regardless of everything else? We really shouldn't have things that only work in one mode.

Thanks!

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EnterpriseDB UK: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

pgadmin-hackers by date:

Previous
From: Dave Page
Date:
Subject: pgAdmin 4 commit: Update cast icons with improved design.
Next
From: Surinder Kumar
Date:
Subject: Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode