Skip to content

Conversation

@jcandkk
Copy link
Contributor

@jcandkk jcandkk commented Nov 5, 2024

Changed the title and link for MAX32690EVKIT web page (https://www.analog.com/en/design-center/evaluation-hardware-and-software/evaluation-boards-kits/MAX32690EVKIT.html)

to point to the MAX32690 solution center on the developer portal (developer.analog.com/solutions/max32690) the solution center has links to the product specific page and offers more content for developers.

@zephyrbot zephyrbot added the platform: ADI Analog Devices, Inc. label Nov 5, 2024
@github-actions
Copy link

github-actions bot commented Nov 5, 2024

Hello @jcandkk, and thank you very much for your first pull request to the Zephyr project!
Our Continuous Integration pipeline will execute a series of checks on your Pull Request commit messages and code, and you are expected to address any failures by updating the PR. Please take a look at our commit message guidelines to find out how to format your commit messages, and at our contribution workflow to understand how to update your Pull Request. If you haven't already, please make sure to review the project's Contributor Expectations and update (by amending and force-pushing the commits) your pull request if necessary.
If you are stuck or need help please join us on Discord and ask your question there. Additionally, you can escalate the review when applicable. 😊

Copy link
Contributor

Choose a reason for hiding this comment

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

I think it make sense to address developer portal.
It requires one more click to reach the board page, but it is fine for me.

This update should be propagated to MAX32690FTHR and apard32690 board too.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you for reviewing and evaluating the documentation link request. I'll also update the other pages as well. Disclaimer: Still learning here - tips and advice appreciated!

Copy link
Member

@MaureenHelm MaureenHelm left a comment

Choose a reason for hiding this comment

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

Same issue as #81146 (review)

@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 60 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 14 days. Note, that you can always re-open a closed pull request at any time.

Updated the reference to the ADI Developer Portal Solution Center with
URL. The Solution Center provides more resources than the product
information. Users can still find the product information from the new
portal link.

Signed-off-by: Janel Kratky <[email protected]>
@MaureenHelm
Copy link
Member

Amended the commit message to pass the compliance check and rebased to latest main

@MaureenHelm MaureenHelm removed the Stale label Jan 23, 2025
@kartben kartben merged commit d6328ec into zephyrproject-rtos:main Jan 24, 2025
18 checks passed
@github-actions
Copy link

Hi @jcandkk!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge.

Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁

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

Labels

platform: ADI Analog Devices, Inc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants