Skip to content
This repository has been archived by the owner on Feb 14, 2025. It is now read-only.

Commit

Permalink
Merge pull request #20 from thoughtspot/feat/added-version-support-fo…
Browse files Browse the repository at this point in the history
…r-charts

added column level settings in custom bar chart and added versioning …
  • Loading branch information
harshmeetTS authored Oct 10, 2024
2 parents 241db56 + 802c9a5 commit 8fa30f0
Show file tree
Hide file tree
Showing 13 changed files with 1,310 additions and 535 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/static.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ name: Deploy static content to Pages
on:
# Runs on pushes targeting the default branch
push:
branches: ["main"]
branches: ['main']

# Allows you to run this workflow manually from the Actions tab
workflow_dispatch:
Expand All @@ -18,7 +18,7 @@ permissions:
# Allow only one concurrent deployment, skipping runs queued between the run in-progress and latest queued.
# However, do NOT cancel in-progress runs as we want to allow these production deployments to complete.
concurrency:
group: "pages"
group: 'pages'
cancel-in-progress: false

jobs:
Expand Down
1 change: 1 addition & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,4 @@ lib
.DS_Store
src/.DS_Store
dist
.vercel
41 changes: 9 additions & 32 deletions ReadMe.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,35 +9,12 @@ npm run dev

### List of urls

```
[
{
"name": "bar-chart-with-measure-names-values",
"sourceUrl": "http://localhost:3002/v1/bar-chart-with-measure-names-values/"
},
{
"name": "combo-chart",
"sourceUrl": "http://localhost:3002/v1/combo-chart/"
},
{
"name": "custom-bar-chart",
"sourceUrl": "http://localhost:3002/v1/custom-bar-chart/"
},
{
"name": "custom-bar-react",
"sourceUrl": "http://localhost:3002/v1/custom-bar-react/"
},
{
"name": "gantt",
"sourceUrl": "http://localhost:3002/v1/gantt/"
},
{
"name": "sunburst",
"sourceUrl": "http://localhost:3002/v1/sunburst/"
},
{
"name": "tester-chart",
"sourceUrl": "http://localhost:3002/v1/tester-chart/"
}
]
```
will be available here [Generate Urls](./generated_urls.json)

### Versioning of charts based on TS

We have introduced new folders for separate versions the folder structure apart from v1, which will contain the current charts code.

- If a minimum version required to run the chart is 10.5 the path to chart code is `src/v10_5/<customcharttype-folder>`
- We would need to update `sourceFolders` array in `vite.config.ts` for deployment.
- The generated urls will look like http://localhost:3002/v10_5/custom-bar-chart/ for TS version 10.5 where custom-bar-chart is the folder where custom bar chart code is present.
6 changes: 5 additions & 1 deletion generated_urls.json
Original file line number Diff line number Diff line change
Expand Up @@ -26,5 +26,9 @@
{
"name": "tester-chart",
"sourceUrl": "http://localhost:3002/v1/tester-chart/"
},
{
"name": "custom-bar-chart",
"sourceUrl": "http://localhost:3002/v10_5/custom-bar-chart/"
}
]
]
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@
"vite-plugin-react": "^3.0.2"
},
"dependencies": {
"@thoughtspot/ts-chart-sdk": "0.0.2-alpha.16",
"@thoughtspot/ts-chart-sdk": "0.0.2-alpha.17",
"chart.js": "3.5.0",
"chartjs-plugin-datalabels": "2.2.0",
"eslint": "8.40.0",
Expand Down
116 changes: 58 additions & 58 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 8fa30f0

Please sign in to comment.