Skip to content

Conversation

zitherean
Copy link
Collaborator

Description
Removed the Facebook link in the website footer.

How to test
Follow these steps to verify this PR works as intended:

Checklist
Done? Check if you have it all in place using this list: (mark with x if done)

  • Contains one of the PR categories in the name
  • Describes changes in the format above
  • Links to an existing issue or discussion
  • Has a "How to test" section

@github-project-automation github-project-automation bot moved this to Review for relevance in Newsletter content Sep 5, 2025
Copy link
Contributor

@brianpeiris brianpeiris left a comment

Choose a reason for hiding this comment

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

Looks good to me.

Before:
Screenshot from 2025-09-05 15-39-01

After:
Screenshot from 2025-09-05 15-38-13

</div>
<div class="flex-1">
<a href="https://github.com/hedyorg/hedy" data-cy="github_button">{{_('hedy_on_github')}}</a><br>
<a href="https://www.facebook.com/profile.php?id=61578101102215" data-cy="facebook_button">{{_('facebook_page')}}</a><br>
Copy link
Member

Choose a reason for hiding this comment

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

Changes look great! Can you fix the conflicts with main? An easy way to do this is to just accept all incoming, and be done with it. Id do it myself but since its a fork its a bit trickier. Thanks!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I don't know if this is what you meant Jesus? I just merged my PR with everything from main. Let me know if there is an issue.

@MarleenGilsing MarleenGilsing moved this from Review for relevance to Not newsletter material in Newsletter content Sep 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: In Progress
Status: Not newsletter material

Development

Successfully merging this pull request may close these issues.

7 participants