Skip to content

Commit

Permalink
Remove Segment analytics (#3486)
Browse files Browse the repository at this point in the history
  • Loading branch information
imnasnainaec authored Dec 18, 2024
1 parent 5251099 commit e6eb771
Show file tree
Hide file tree
Showing 17 changed files with 248 additions and 42,697 deletions.
1 change: 0 additions & 1 deletion .env.local.template

This file was deleted.

11 changes: 2 additions & 9 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -153,16 +153,9 @@ A rapid word collection tool. See the [User Guide](https://sillsdev.github.io/Th
- `COMBINE_SMTP_ADDRESS`
- `COMBINE_SMTP_FROM`

2. _(Optional)_ To opt in to segment.com analytics to test the analytics during development:
2. Run `npm start` from the project directory to install dependencies and start the project.

```bash
# For Windows, use `copy`.
cp .env.local.template .env.local
```

3. Run `npm start` from the project directory to install dependencies and start the project.

4. Consult our [C#](docs/style_guide/c_sharp_style_guide.md) and [TypeScript](docs/style_guide/ts_style_guide.md) style
3. Consult our [C#](docs/style_guide/c_sharp_style_guide.md) and [TypeScript](docs/style_guide/ts_style_guide.md) style
guides for best coding practices in this project.

[chocolatey]: https://chocolatey.org/
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,6 @@ spec:
configMapKeyRef:
key: SERVER_NAME
name: env-frontend
{{- if .Values.configAnalyticsWriteKey }}
- name: CONFIG_ANALYTICS_WRITE_KEY
valueFrom:
configMapKeyRef:
key: CONFIG_ANALYTICS_WRITE_KEY
name: env-frontend
{{- end }}
ports:
- containerPort: 80
- containerPort: 443
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,3 @@ data:
CONFIG_OFFLINE: {{ .Values.configOffline | quote | lower }}
CONFIG_EMAIL_ENABLED: {{ .Values.global.emailEnabled | quote | lower }}
CONFIG_SHOW_CERT_EXPIRATION: {{ .Values.configShowCertExpiration | quote | lower }}
{{- if .Values.configAnalyticsWriteKey }}
CONFIG_ANALYTICS_WRITE_KEY: {{ .Values.configAnalyticsWriteKey | quote }}
{{- end }}
1 change: 0 additions & 1 deletion deploy/helm/thecombine/charts/frontend/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ imageName: combine_frontend

# The additional domain list is a space-separated string list of domains
combineAddlDomainList: ""
configAnalyticsWriteKey: ""
configCaptchaSiteKey: ""
configOffline: false
configShowCertExpiration: false
1 change: 0 additions & 1 deletion deploy/scripts/setup_files/profiles/prod.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ charts:
enabled: false
# Frontend configuration items:
frontend:
configAnalyticsWriteKey: "j9EeK4oURluRSIKbaXCBKBxGCnT2WahB"
configCaptchaSiteKey: "0x4AAAAAAAiMciPlBW1aA1iL" # Turnstile site key
configShowCertExpiration: false
# Maintenance configuration items
Expand Down
1 change: 0 additions & 1 deletion deploy/scripts/setup_files/profiles/staging.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ charts:
thecombine:
# Frontend configuration items:
frontend:
configAnalyticsWriteKey: "AoebaDJNjSlOMRUH87EaNjvwkQpfLoyy"
configCaptchaSiteKey: "0x4AAAAAAAiMciPlBW1aA1iL" # Turnstile site key
configShowCertExpiration: false
global:
Expand Down
39,542 changes: 119 additions & 39,423 deletions docs/user_guide/assets/licenses/frontend_licenses.txt

Large diffs are not rendered by default.

1 change: 0 additions & 1 deletion nginx/init/25-combine-runtime-config.sh
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,6 @@ env_map=(
["CONFIG_USE_CONNECTION_URL"]="useConnectionBaseUrlForApi"
["CONFIG_CAPTCHA_REQUIRED"]="captchaRequired"
["CONFIG_CAPTCHA_SITE_KEY"]="captchaSiteKey"
["CONFIG_ANALYTICS_WRITE_KEY"]="analyticsWriteKey"
["CONFIG_OFFLINE"]="offline"
["CONFIG_EMAIL_ENABLED"]="emailServicesEnabled"
["CONFIG_SHOW_CERT_EXPIRATION"]="showCertExpiration"
Expand Down
Loading

0 comments on commit e6eb771

Please sign in to comment.