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

Change NEW announcement to point to the Engine video post #325

Closed
campoy opened this issue Nov 14, 2018 · 4 comments · Fixed by #327
Closed

Change NEW announcement to point to the Engine video post #325

campoy opened this issue Nov 14, 2018 · 4 comments · Fixed by #327

Comments

@campoy
Copy link
Contributor

campoy commented Nov 14, 2018

I would like to update the announcement banner on the landing from PGA (shown below)

screen shot 2018-11-14 at 12 11 56 pm

to a link to the blog post we wrote presenting the video for the engine.

The announcement should read something like "Watch a 5 minutes video to understand how the engine works" and point to https://medium.com/sourcedtech/source-d-engine-in-5-minutes-6b3b19789594.

@dpordomingo
Copy link
Contributor

dpordomingo commented Nov 14, 2018

(a) Should the NEW section retrieve content automatically from somewhere (like «latest updates» or «job offers» section)?
(b) Or is that section static and should it be edited > PR > released/deployed?

If it's (a), from where?
If it's (b) —I'd vote for it— it's easier: I'd load its contents from a new subkey under hugo.config.yaml::params

  new:
    title: #link text
    url: #link url

quick and dirty, and it would make the changes easier to be done, reviewed and merged.
(An alternative to that config file could be to create a new data file hugo/data/news.yml with the same purpose)

@campoy
Copy link
Contributor Author

campoy commented Nov 16, 2018

I'm happy with b, as long as it's easy to change over time.

@dpordomingo
Copy link
Contributor

ready to be merged/deployed #327 ;)

@dpordomingo
Copy link
Contributor

already at prod
💃

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

Successfully merging a pull request may close this issue.

2 participants