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

Make it easier to get started with OpenAMP #103

Merged
merged 4 commits into from
Jul 13, 2023

Conversation

nathalie-ckc
Copy link
Collaborator

Address the suggestions in issue #97 and also fix the jumbotron button to point to the GitHub organization instead of just the OpenAMP library.

Updated the "View on Github" link in jumbotron to point to the OpenAMP organization, instead of just the library.
Added Getting Started section to footer that points to documentation, demos, mailing list, and Discord.
Added social media icons that point to our GitHub, LinkedIn, and YouTube.
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

Copy link

@wmamills wmamills left a comment

Choose a reason for hiding this comment

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

My only question is about the discord link. I don't know what that looks like if you don't have discord running and not already a member of the channel & server.

I presume the user can exit w/o doing anything. I tried to test in incognito but it still redirected to me local discord app.

I presume this is OK as is.

@wmamills
Copy link

Ok I tested the discord link using a dummy Linux account. It is fine.

Copy link

@wmamills wmamills left a comment

Choose a reason for hiding this comment

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

Approved

@nathalie-ckc nathalie-ckc merged commit 1f0c4ae into OpenAMP:master Jul 13, 2023
2 checks passed
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