From: | Dave Page <dpage(at)pgadmin(dot)org> |
---|---|
To: | Surinder Kumar <surinder(dot)kumar(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode |
Date: | 2017-09-13 11:06:50 |
Message-ID: | CA+OCxoww3angrAPh+9wo4yq6un4h35Ht1ELzO-vRV9Kwnc5ZXg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Wed, Sep 13, 2017 at 10:34 AM, Surinder Kumar <
surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
> On Wed, Sep 13, 2017 at 2:52 PM, Dave Page <dpage(at)pgadmin(dot)org> wrote:
>
>>
>>
>> On Wed, Sep 13, 2017 at 10:06 AM, Surinder Kumar <
>> surinder(dot)kumar(at)enterprisedb(dot)com> wrote:
>>
>>>
>>>
>>> On Wed, Sep 13, 2017 at 2:18 PM, Dave Page <dpage(at)pgadmin(dot)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
From | Date | Subject | |
---|---|---|---|
Next Message | Surinder Kumar | 2017-09-13 12:26:45 | Re: [pgAdmin4][Patch]: Run MakeFile to generate builds in production mode |
Previous Message | Dave Page | 2017-09-13 09:43:44 | pgAdmin 4 commit: Update cast icons with improved design. |