You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by "rusackas (via GitHub)" <gi...@apache.org> on 2025/03/14 20:07:29 UTC
[PR] feat(docs): Adding Kapa.ai integration [superset]
rusackas opened a new pull request, #32682:
URL: https://github.com/apache/superset/pull/32682
<!---
Please write the PR title following the conventions at https://www.conventionalcommits.org/en/v1.0.0/
Example:
fix(dashboard): load charts correctly
-->
### SUMMARY
Adds Kapa.ai chat bot integration to docs website! I'll probably make some color/logo and title tweaks, but it's working (and ASF-approved!)
<img width="1629" alt="image" src="https://github.com/user-attachments/assets/d02ac93a-e922-4e17-8625-43d284a3c377" />
<img width="1632" alt="image" src="https://github.com/user-attachments/assets/ff221bc7-d6c9-438b-b02d-27f985df850b" />
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
<!--- Skip this if not applicable -->
### TESTING INSTRUCTIONS
<!--- Required! What steps can be taken to manually verify the changes? -->
### ADDITIONAL INFORMATION
<!--- Check any relevant boxes with "x" -->
<!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] Migration is atomic, supports rollback & is backwards-compatible
- [ ] Confirm DB migration upgrade and downgrade tested
- [ ] Runtime estimates and downtime expectations provided
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] feat(docs): Adding Kapa.ai integration [superset]
Posted by "hainenber (via GitHub)" <gi...@apache.org>.
hainenber commented on PR #32682:
URL: https://github.com/apache/superset/pull/32682#issuecomment-2726242355
I think this will help tremendously with questions people have on Superset usage :D
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] feat(docs): Adding Kapa.ai integration [superset]
Posted by "hainenber (via GitHub)" <gi...@apache.org>.
hainenber merged PR #32682:
URL: https://github.com/apache/superset/pull/32682
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] feat(docs): Adding Kapa.ai integration [superset]
Posted by "korbit-ai[bot] (via GitHub)" <gi...@apache.org>.
korbit-ai[bot] commented on code in PR #32682:
URL: https://github.com/apache/superset/pull/32682#discussion_r1996217693
##########
docs/docusaurus.config.ts:
##########
@@ -333,6 +333,31 @@ const config: Config = {
// src: 'https://www.bugherd.com/sidebarv2.js?apikey=enilpiu7bgexxsnoqfjtxa',
// async: true,
// },
+ '/script/matomo.js',
Review Comment:
### Missing Analytics Script Documentation <sub></sub>
<details>
<summary>Tell me more</summary>
###### What is the issue?
The matomo.js script inclusion lacks documentation about its purpose and impact on the website.
###### Why this matters
Without documentation, future maintainers won't understand why this analytics script is included or what data it collects.
###### Suggested change ∙ *Feature Preview*
// Matomo analytics tracking script for monitoring website usage and user behavior
'/script/matomo.js',
</details>
<sub>
[](https://app.korbit.ai/feedback/aa91ff46-6083-4491-9416-b83dd1994b51/175c4214-1089-4769-8e5b-187bdbeb3b06?suggestedFixEnabled=true)
💬 Looking for more details? Reply to this comment to chat with Korbit.
</sub>
<!--- korbi internal id:9ffd3751-d02f-4917-bdb6-8b5343257c17 -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] feat(docs): Adding Kapa.ai integration [superset]
Posted by "mistercrunch (via GitHub)" <gi...@apache.org>.
mistercrunch commented on PR #32682:
URL: https://github.com/apache/superset/pull/32682#issuecomment-2730081154
I don't see the chatbot enabled post-merge, do we need to do something else to get it up and running?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org
Re: [PR] feat(docs): Adding Kapa.ai integration [superset]
Posted by "rusackas (via GitHub)" <gi...@apache.org>.
rusackas commented on PR #32682:
URL: https://github.com/apache/superset/pull/32682#issuecomment-2741253692
Missed the comment above, but in short, there were some CSP edits that had to be made. Should be all good as of now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org