这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@ajeetdsouza
Copy link
Contributor

@ajeetdsouza ajeetdsouza commented Jul 1, 2019

Description

The server and cli currently do not pass the server version to the rendered console. This PR adds the serverVersion to the rendered template.

Affected components

  • Server
  • CLI

Related Issues

#2444

Solution and Design

Added serverVersion to the console.html template file served by the cli and server modules.

Steps to test and verify

  • Serve the console via server/cli. serverVersion should be available in a <script> tag.

Limitations, known bugs & workarounds

None

To the Reviewer (cli)

  • No new functions/modules have been added
    Critical files:
  • cli/assets/unversioned/console.html: added serverVersion under window.__env
  • cli/assets/v1.0/console.html: added serverVersion under window.__env
  • cli/commands/console.go: pass serverVersion to console.html template

To the Reviewer (server)

  • No new functions/modules have been added
  • server/src-rsr/console.html: added serverVersion under window.__env
  • server/src-lib/Hasura/Server/App.hs: pass serverVersion to console.html template

@netlify
Copy link

netlify bot commented Jul 1, 2019

Deploy preview for hasura-docs ready!

Built with commit 6fcaf07

https://deploy-preview-2452--hasura-docs.netlify.com

@ajeetdsouza ajeetdsouza added c/cli Related to CLI c/server Related to server s/do-not-merge Do not merge this pull request to master labels Jul 1, 2019
@hasura-bot
Copy link
Contributor

Review app for commit 393d40c deployed to Heroku: https://hge-ci-pull-2452.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2452-393d40ca

@shahidhk shahidhk changed the title Pass serverVersion to console.html template add server version to console html (close #2444) Jul 2, 2019
@ajeetdsouza ajeetdsouza removed the s/do-not-merge Do not merge this pull request to master label Jul 2, 2019
Copy link
Member

@shahidhk shahidhk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hasura-bot
Copy link
Contributor

Review app for commit 6fcaf07 deployed to Heroku: https://hge-ci-pull-2452.herokuapp.com
Docker image for server: hasura/graphql-engine:pull2452-6fcaf07a

@rikinsk-zz
Copy link

rikinsk-zz commented Jul 2, 2019

We can use the same PR to make the relevant console changes as well

Dont want to block this. Console changes can happen later

@rikinsk-zz rikinsk-zz added c/console Related to console and removed c/console Related to console labels Jul 2, 2019
@shahidhk shahidhk merged commit b184ce0 into hasura:master Jul 2, 2019
@hasura-bot
Copy link
Contributor

Review app https://hge-ci-pull-2452.herokuapp.com is deleted

@hasura-bot
Copy link
Contributor

Beep boop! 🤖

GIF

Awesome work @ajeetdsouza! All of us at Hasura ❤️ what you did.

Thanks again 🤗

wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 3, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 4, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 8, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 12, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 23, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 24, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
wawhal added a commit to wawhal/graphql-engine that referenced this pull request Jul 24, 2019
* render 404 for nonexistent resources (hasura#2341)

* update console dev logging + add logging disable instructions to README (hasura#2388)

* update table rename handling (hasura#2393)

* suggest column default values (hasura#2352)

* fix postgres query to fetch pg_types default functions (close hasura#2399) (hasura#2402)

* add graphql-voyager to console (close hasura#314) (hasura#2369)

* clear header state for remote-schema (close hasura#2396) (hasura#2401)

* change browse rows action buttons heading id (close hasura#2403) (hasura#2413)

* created docker compose for build and launch of tutorial (hasura#2406)

* fix renaming of columns used in insert permission (fix hasura#2398) (hasura#2414)

* graphql2chartjs: fix mapping of non-array types (hasura#2307)

* add name of ci environment to telemetry data (hasura#2382)

* learn: update react native tutorial with videos (hasura#2390)

* create down migration when a column is dropped (close hasura#2291) (hasura#2385)

* add new column row during add table only after completed prev column def (hasura#2428)

* add a make rule to build server docker image locally (hasura#2410)

* learn: fix twitter share image on homepage (hasura#2437)

* add note on exporting multiple schemas as migration (hasura#2436)

* fix browse rows query error handling + boolean filter value (close hasura#2421) (hasura#2440)

* graphql2chartjs: bump version to 0.3.0 (hasura#2445)

* fix to show error notification when run sql fails in cli mode (close hasura#2438) (hasura#2446)

* freeze requirements.txt for server tests (hasura#2450)

* update postgres permission documentation (fix hasura#2404) (hasura#2405)

* refactor AddTable validation on console (close hasura#1712) (hasura#2408)

* change default type placeholder from 'text' to type (fix 2394) (hasura#2458)

* add server version to console html (close hasura#2444) (hasura#2452)

* fix type placeholders in console insert/edit row (hasura#2460)

* fix relationship page error if array fk relationship exists with table in another schema (hasura#2459)

* infer relationship type using uniqueness constraint of foreign key (close hasura#969) (hasura#2347)

* use server version from console html (hasura#2464)

* add schema explorer ui

* wire up create API, WIP

* ui fixes

* integrate with proxy API

* fetch remote relationships in loadSchema

* fixes

* finish up UI for remote rels; waiting on edit API

* break relationships ui into sections

* move graphql schema to local state

* update relationship label

* restructure

* update styles; add tooltips

* filter out fields (scalar type || wrapped scalar type) and do accept any args

* remove an unnecessary log

* version check init

* add version check for remote relationship on loadSchema

* use FT_REMOTE_RELATIONSHIPS

* allow adding string values for scalar args

* comment code

* add update relationship feature

* cover edge cases for update remote rel

* fix semver issue where sometimes the query would never be called

* remove unnecessary fields from schema tree

* stable

* wip

* wip

* wip

* add version checks

* invalidate remote schema cache appropriately

* Remove unnecessary logs
polRk pushed a commit to polRk/graphql-engine that referenced this pull request Feb 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c/cli Related to CLI c/server Related to server

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants