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

Generate open-graph images for blog posts #30

Merged
merged 11 commits into from May 22, 2023
Merged

Generate open-graph images for blog posts #30

merged 11 commits into from May 22, 2023

Conversation

pepicrft
Copy link
Owner

Context

All the routes in this website have the same static image as an open graph image. I'd like to make it more dynamic based on the content of the URL being shared.

What

I'm adding dynamic open-graph generation for the blog posts in the website. The way I'm approaching the problem is by:

  • Having a Mix task, mix phx.gen.open_graph, that runs as part of deployment to generate the images for all the blog posts.
  • Having a watcher in development that changes the image when a blog post changes.

@pepicrft pepicrft self-assigned this May 21, 2023
@@ -1 +1,3 @@
use "nix"
export ERL_AFLAGS="-kernel shell_history enabled -kernel shell_history_file_bytes 1024000"
Copy link
Owner Author

Choose a reason for hiding this comment

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

I added this one to preserve the shell history across the runs of IEx.

{:ok, %{watcher_pid: watcher_pid}}
end

def handle_info({:file_event, watcher_pid, {path, events}}, %{watcher_pid: watcher_pid} = state) do
Copy link
Owner Author

Choose a reason for hiding this comment

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

This function is not being called at all.

@pepicrft pepicrft merged commit 79d3bf4 into main May 22, 2023
1 check passed
@pepicrft pepicrft deleted the open-graph-images branch May 22, 2023 08:42
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.

None yet

1 participant