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

fix: rm twitter inset since non-operational #1429

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

wdconinc
Copy link
Contributor

@wdconinc wdconinc commented Jul 6, 2023

This removes the twitter inset that doesn't show anything useful due to twitter policy changes (see below). We could replace it with a link to the twitter profile and other social profiles, but I don't think that would necessarily fit with the layout here.

image

@graeme-a-stewart graeme-a-stewart merged commit 784eed3 into main Jul 6, 2023
@graeme-a-stewart
Copy link
Member

Thanks @wdconinc - let's just disable it for now. As we are rethinking the website we can see where social media links best go in the future.

@wdconinc wdconinc deleted the fix-rm-twitter-inset branch July 6, 2023 14:07
@klieret
Copy link
Member

klieret commented Jul 7, 2023

Oh wow, that's too bad. Do you have any link for the policy changes, I wonder what this is about?

@wdconinc
Copy link
Contributor Author

wdconinc commented Jul 7, 2023

@klieret
Copy link
Member

klieret commented Jul 7, 2023

thanks. This is probably something slightly different on top of this (because I can't see the tweets despite being logged in), but this really impacts the usefulness and I agree we should remove it at this poitn

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

Successfully merging this pull request may close these issues.

3 participants