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

Avocado Website Update #28

Open
wants to merge 2 commits into
base: new_website
Choose a base branch
from

Conversation

harvey0100
Copy link

Avocado website updated according to designs provided lots of new content such as updated utils table.
Many changes to the Main Avocado Framework Page included. Other repos such as Aexpect / VT links point to their repo.

Avocado website updated according to designs provided
lots of new content such as updated utils table.
Many changes to the Main Avocado Framework Page included.
Other repos such as Aexpect / VT links point to their repo.

Signed-off-by: Harvey Lynden <[email protected]>
Copy link
Contributor

@richtja richtja left a comment

Choose a reason for hiding this comment

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

Hi @harvey0100, thank you very much for this, the new page design is awesome. I wasn't able to properly review the overall implementation because I don't have enough experience with web development. But I have few practical comments related to future maintainability of this page. Also, can you please test this with current build process to make sure that this will be properly deployed after merge. Thanks

CREDITS Show resolved Hide resolved
LICENSE Outdated
@@ -1,116 +0,0 @@
CC0 1.0 Universal
Copy link
Contributor

Choose a reason for hiding this comment

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

We need to keep the LICENSE.

_config.yml Outdated Show resolved Hide resolved
_data/versions.yml Outdated Show resolved Hide resolved
_data/modules.yml Outdated Show resolved Hide resolved
resources/images/cleber.jpg Outdated Show resolved Hide resolved
@harvey0100 harvey0100 force-pushed the new_website branch 2 times, most recently from 614bbe4 to 948828c Compare October 10, 2024 15:37
Added License, Credits to designers.
Deployment script added for workflow to
fetch avatars for the autils table.

Signed-off-by: Harvey Lynden <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Review Requested
Development

Successfully merging this pull request may close these issues.

2 participants