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.

Sign up for GitHub

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

Jump to bottom

Update Developer site logos #8357

Merged
merged 3 commits into from
Jan 8, 2025
Merged

Update Developer site logos #8357

merged 3 commits into from
Jan 8, 2025

Conversation

fletcherdodd
Copy link
Contributor

No description provided.

@fletcherdodd fletcherdodd added update site Issue/PR updates information about a site in the repo. multiple sites Issue/PR includes multiple sites. labels Jan 6, 2025
hkamran80
hkamran80 reviewed Jan 6, 2025
View reviewed changes
Copy link
Member

@hkamran80 hkamran80 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These SVGs are with all of SVGOMG's features turned on (except for remove XMLNS) and the precisions set to 1 (number) and 0 (transform).

img/a/alchemy.com.svg Outdated Show resolved Hide resolved
img/g/goteleport.com.svg Outdated Show resolved Hide resolved
fletcherdodd and others added 2 commits January 8, 2025 14:25
@fletcherdodd @hkamran80
Fix Alchemy SVG
43ccb76 
Co-authored-by: H. Kamran <hkamran@hkamran.com>
Signed-off-by: Fletcher Dodd <fletcherdodd@outlook.com>
@fletcherdodd @hkamran80
Fix Teleport SVG
2e47a3e 
Co-authored-by: H. Kamran <hkamran@hkamran.com>
Signed-off-by: Fletcher Dodd <fletcherdodd@outlook.com>
@fletcherdodd
Copy link
Contributor Author

These SVGs are with all of SVGOMG's features turned on (except for remove XMLNS) and the precisions set to 1 (number) and 0 (transform).

Ooops my cookies on the online verison of it must have been reset :)

@Carlgo11 Carlgo11 merged commit bd55345 into 2factorauth:master Jan 8, 2025
2 checks passed
@fletcherdodd fletcherdodd deleted the developer branch January 8, 2025 23:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hkamran80 hkamran80 hkamran80 left review comments

@Carlgo11 Carlgo11 Carlgo11 approved these changes

Assignees
No one assigned
Labels
multiple sites Issue/PR includes multiple sites. update site Issue/PR updates information about a site in the repo.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fletcherdodd @Carlgo11 @hkamran80