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

Add blog post for System Reference webinar #94

Merged
merged 4 commits into from
Feb 27, 2023

Conversation

nathalie-ckc
Copy link
Collaborator

2022-Dec-13 OpenAMP System Reference working group webinar materials are now posted on the Linaro resource page. This blog post shares that material.

nathalie-ckc and others added 3 commits January 18, 2023 10:12
2022-Dec-13 OpenAMP System Reference working group webinar materials are now posted on the Linaro resource page. This blog post shares that material.
@nathalie-ckc
Copy link
Collaborator Author

nathalie-ckc commented Jan 18, 2023

Hi @arnopo and @wmamills ,
I have posted the webinar materials. Would be great to have a second pair of eyes check to make sure all the relevant info is there. So, we can merge after one of you looks at it (don't have to wait for both), unless there are changes to discuss.
You can go to "Show All Checks" then go to the "Deploy preview" > "Details" and go to the News & Blogs section & view the latest blog post there in the GitHub Actions staging area.
Thanks & regards,
Nathalie

Copy link
Contributor

@arnopo arnopo left a comment

Choose a reason for hiding this comment

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

LGTM with a suggestion on discord link.
I also don't understand the last commit: "Merge branch 'OpenAMP:master' into blog_systref_webinar"



The Linaro resource page with the video of the presentation and the slides can be found [here](https://resources.linaro.org/en/resource/2QSzYLK9Uxq4CFhVBYAnvb).
In the posted slides are links to the documentation and repositories for reproducing the demos, as well as a link to join the OpenAMP community Discord channel.
Copy link
Contributor

Choose a reason for hiding this comment

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

What about adding the link to the the Dicsord channel in the "Useful Links" on https://www.openampproject.org/ page?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@arnopo , per Bill's suggestion I'll make it a different PR to update the Useful Links. I will file issues for these items so I don't forget.
The last commit was required because the GitHub Actions run a check for broken links. CHAOSS keeps moving their code of conduct around. I updated to the new link in PR #95 and merged that without review. The last commit pulls that change in so that GitHub Actions for this PR pass. 8825c71 is the diff.

@wmamills
Copy link

wmamills commented Jan 19, 2023

The blog post looks OK to me.

You have to go looking for it. I wonder if the front page should automatically include the latest blog post (unless it is > 6 months old). If we do this it should be a different PR.

If we add the discord link it should be a different PR.

The resource hub gives access to the video but it is 2 hours long. I was hopping to add a TOC w/ timestamps and links but I don't know if that is possible on the resource hub. We could do that at Youtube or we could add individual videos. I also wondered if we should have a direct link to the PDF.

I am OK merging as is or edit with new links, whichever you all think is better.

Bill pointed out that it's hard to notice the hyperlink on here, because it is just 1 word.  Hyperlink more text.
@wmamills
Copy link

Yes this looks good. I did not realize this was still waiting on me.

@arnopo arnopo merged commit 542882e into OpenAMP:master Feb 27, 2023
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

3 participants