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

Replace 'Infi' with something more generic in topic texts #60

Merged

Conversation

suzaninfi
Copy link
Contributor

The word 'Infi' has been replaced with something more generic in the topic texts. Fixes issue #40

@suzaninfi suzaninfi added the content Changes to the actual content of The Infi Way label May 10, 2023
@netlify
Copy link

netlify bot commented May 10, 2023

Deploy Preview for the-infi-way ready!

Name Link
🔨 Latest commit b9bb86f
🔍 Latest deploy log https://app.netlify.com/sites/the-infi-way/deploys/645b458df8cabc0008524fa2
😎 Deploy Preview https://deploy-preview-60--the-infi-way.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@jeroenheijmans
Copy link
Member

LGTM. For a second I thought there were a few others as well to tackle (e.g. "The Infi Test" at the top) but I suppose those should be tackled in #41

Preview seems good so I'll merge this one.

@jeroenheijmans jeroenheijmans merged commit 1fa70ce into main May 16, 2023
4 checks passed
@jeroenheijmans jeroenheijmans deleted the 40-replace-infi-in-texts-with-we-where-reasonable branch May 16, 2023 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Changes to the actual content of The Infi Way
Development

Successfully merging this pull request may close these issues.

None yet

2 participants