Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: gauge mini #366

Closed
wants to merge 14 commits into from
Closed

Conversation

Sciator
Copy link
Contributor

@Sciator Sciator commented Nov 29, 2020

All info (including todos) inside Giraffe PR
influxdb#20404

image

Build influx with custom giraffe,ui

(all this commads are squashed into one down in this description)

  • clone everything into one file

    git clone https://github.com/Sciator/giraffe.git
    cd ./giraffe/
    git checkout feat-new-plot-types
    cd ..
    git clone https://github.com/Sciator/ui.git
    cd ./ui/
    git checkout feat-new-plot-types
    cd ..
    git clone https://github.com/Sciator/influxdb.git
    cd ./influxdb/
    git checkout feat-new-plot-types
    cd ..
    
  • inside giraffe cd ./giraffe/giraffe

    • yarn install - install dependencies
    • yarn build - build giraffe
    • yarn link - create linkable local dependency
    • cd ../.. - leave directory
  • inside ui cd ./ui

    • yarn link @influxdata/giraffe - add dependency to local giraffe
    • yarn build - build UI
    • cd .. - leave directory
  • inside influxdb cd ./influxdb

    • make - build db
    • go run ./cmd/influxd --assets-path=../ui/build --feature-flags=gaugeMini=true - run influxdb with custom ui and gaugeMini enabled
    • influx is now running on localhost:8086

When changing ui, influxdb doen't need to be restarted. Just call yarn build inside ui folder and refresh webpage when done.

all commands on one line - prepare and run all

mkdir new-components && cd new-components && (   git clone https://github.com/Sciator/giraffe.git   & git clone https://github.com/Sciator/ui.git   & git clone https://github.com/Sciator/influxdb.git ) && (   (cd ./giraffe/ && git checkout feat-new-plot-types)   & (cd ./ui/ && git checkout feat-new-plot-types)   & (cd ./influxdb/ && git checkout feat-new-plot-types) ) && (   (     cd ./giraffe/giraffe && yarn install && yarn build && yarn link     &&  cd ../.. && cd ui && yarn link @influxdata/giraffe && yarn build   )   & cd ./influxdb && make && go run ./cmd/influxd --assets-path=../ui/build --feature-flags=gaugeMini=true ) 
  • revert swagger url before merge

@Sciator Sciator changed the title feat: gauge mini WIP feat: gauge mini Dec 23, 2020
@Sciator Sciator changed the title WIP feat: gauge mini feat: gauge mini WIP Dec 29, 2020
@Sciator Sciator changed the title feat: gauge mini WIP feat: gauge mini Dec 30, 2020
@Sciator Sciator marked this pull request as ready for review January 5, 2021 13:18
@Sciator Sciator mentioned this pull request Jan 22, 2021
10 tasks
@ivankudibal ivankudibal requested a review from TCL735 January 22, 2021 12:49
@genehynson
Copy link
Contributor

no plans to put this in the UI currently, but leaving the Giraffe PR open

@genehynson genehynson closed this Jun 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants