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

[website] Update the Careers page role #41384

Merged
merged 3 commits into from
Mar 6, 2024

Conversation

danilo-leal
Copy link
Contributor

Also updated the Base UI blog post to still link to the Careers page so it's a bit open-ended. Or should we remove the link entirely?

@danilo-leal danilo-leal added the website Pages that are not documentation-related, marketing-focused. label Mar 6, 2024
@danilo-leal danilo-leal requested a review from colmtuite March 6, 2024 12:19
@danilo-leal danilo-leal self-assigned this Mar 6, 2024
@mui-bot
Copy link

mui-bot commented Mar 6, 2024

Netlify deploy preview

https://deploy-preview-41384--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against ef83f65

@colmtuite
Copy link
Contributor

colmtuite commented Mar 6, 2024

Or should we remove the link entirely?

I would just remove the link entirely so it's not dilutive.

Also, the page will still be available at /staff-ui-engineer-base-ui/. Not sure if you wanna remove those files too.

@danilo-leal
Copy link
Contributor Author

Yeah, makes sense to remove the hiring mentions on the blog post. I'll keep the job description page files, though, as we may want to just reactivate it in a few months in the future. It's not linked anywhere else in the codebase from this point, so you can only access it if you saved the URL somehow 😬 Guess we're good?!

@danilo-leal danilo-leal merged commit 98d42ba into mui:master Mar 6, 2024
19 checks passed
@danilo-leal danilo-leal deleted the update-careers-page branch March 6, 2024 17:02
cipherlogs pushed a commit to cipherlogs/material-ui that referenced this pull request Mar 8, 2024
mnajdova pushed a commit to mnajdova/material-ui that referenced this pull request Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
website Pages that are not documentation-related, marketing-focused.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants