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

docs: add guide to send Fusionauth data to Segment #3229

Open
wants to merge 25 commits into
base: main
Choose a base branch
from

Conversation

sixhobbits
Copy link
Collaborator

No description provided.

@sixhobbits sixhobbits requested review from a team as code owners August 29, 2024 14:22
@alex-fusionauth alex-fusionauth self-assigned this Sep 6, 2024
@alex-fusionauth alex-fusionauth added the content Alert marketing to this change label Sep 6, 2024
mooreds added a commit that referenced this pull request Sep 17, 2024
mooreds added a commit that referenced this pull request Sep 18, 2024
* removing reference to twilio segment guide because it is not live yet. Here is the PR: #3229

* fix busted links
@alex-fusionauth
Copy link
Contributor

@sixhobbits I have requested new images instead of the mermaid diagrams. I am trying to get all of mermaid removed from the current site and would like to do the same on new PRs.

Could you pull in main again I think it will fix the value runner.

@sixhobbits
Copy link
Collaborator Author

thanks @alex-fusionauth - good to know. Just to clarify, you want us to replace the mermaid diagram with something else? Is this just a format thing, or a style thing -- i.e. can we just use a PNG export from Mermaid still, or are there specific requirements for images?

@sixhobbits sixhobbits force-pushed the ritza/add-twilio-segment-guide branch from 0a6c5d4 to 8955ebc Compare September 27, 2024 10:43
@sixhobbits sixhobbits requested a review from a team as a code owner September 27, 2024 10:43
@alex-fusionauth
Copy link
Contributor

@sixhobbits sorry for the delay on this, we are working towards removing all mermaid diagrams to get more on brand with everything.

Below are the new images, I you could remove mermaid references for these
running-fusionauth-in-docker-2
filtering-web-service-segment-fusionauth

@sixhobbits
Copy link
Collaborator Author

@alex-fusionauth - thanks for these -- is it possible for us to get a copy of the source so we can make small changes? Or if not, it would be great if the arrows and connectors could be updated to consistently always start/end either inside or outside the container boundary or it might cause confusion.

screenshot 2024-10-17 at 14 17 00

@alex-fusionauth
Copy link
Contributor

@SeanBobby I am going to be out for a while, can we get this updated for consistency?

@sixhobbits
Copy link
Collaborator Author

not a huge thing so we will update the post with the ones above so it can be published like that, and we can update the images post publication if necessary

@sixhobbits
Copy link
Collaborator Author

@alex-fusionauth we've added the images here, let me know if you want anything else changed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Alert marketing to this change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants