Document toggle for maximum courses on homepage#37680
Conversation
Added toggle documentation for HOMEPAGE_COURSE_MAX.
|
Thanks for the pull request, @Ozohu1! This repository is currently maintained by Once you've gone through the following steps feel free to tag them in a comment and let them know that your changes are ready for engineering review. 🔘 Get product approvalIf you haven't already, check this list to see if your contribution needs to go through the product review process.
🔘 Provide contextTo help your reviewers and other members of the community understand the purpose and larger context of your changes, feel free to add as much of the following information to the PR description as you can:
🔘 Get a green buildIf one or more checks are failing, continue working on your changes until this is no longer the case and your build turns green. DetailsWhere can I find more information?If you'd like to get more details on all aspects of the review process for open source pull requests (OSPRs), check out the following resources: When can I expect my changes to be merged?Our goal is to get community contributions seen and reviewed as efficiently as possible. However, the amount of time that it takes to review and merge a PR can vary significantly based on factors such as:
💡 As a result it may take up to several weeks or months to complete a review and merge your PR. |
|
Hi team, |
Hi @Ozohu1! We'll get the tests running for you, and the CLA check should hopefully turn green. Please note, that in the U.S. we have been out of the office today, Fri. Nov. 28, and yesterday, Thurs. Nov. 27 for holidays. Please check back early next week as the CLA process might have been slightly delayed due to the holidays. |
|
Hi team, |
|
Hi team, |
|
Good day Team. I am writing to follow up on the signed CLA approval. kindly approve the next steps. Thank you |
Hi @Ozohu1! Your CLA is all set, and I have a note into our team to get the checks running for you. Thanks for your patience! |
|
@Ozohu1 |
|
@Ozohu1 There are also some failing Pylint checks, so please have a look at those as well. |
Flagging this for you @Ozohu1 |
|
Closing for now due to inactivity. |
|
Hello, I would like to continue working on this contribution and address the outstanding feedback regarding the checks and commit requirements. If this documentation update is still relevant, I would be grateful if the pull request could be reopened so I can make the necessary updates and proceed with the review process. Thank you for your time and support. |
Hello, I would like to continue working on this contribution and address the outstanding feedback regarding the checks and commit requirements. If this documentation update is still relevant, I would be grateful if the pull request could be reopened so I can make the necessary updates and proceed with the review process. Thank you for your time and support. |
|
Hi @Ozohu1 - thanks for the ping. This is reopened. We will get the checks enabled for you. |
Hi @mphilbrick211, thank you for reopening this. I have updated the branch with the latest base changes. I can see the workflows are now awaiting maintainer approval. Please let me know if there is anything else you need from me at this stage. Thank you. |
Hello @mphilbrick211. Thanks for reopening. I am currently blocked and not able to update the branch. Could you please help me with the current blockers so I can proceed with my contribution? Thanks for your prompt response |
|
Hello all. I am currently blocked and not able to update the branch. Could you please help me with the blockers so I can proceed with the next steps? Thanks for your prompt response |
Added toggle documentation for HOMEPAGE_COURSE_MAX.
Description
This pull request adds missing documentation for the HOMEPAGE_COURSE_MAX toggle.
The new comment explains what the setting does and how it affects the number of courses displayed on the LMS homepage.
Supporting Information
The change only updates documentation.
No functional code behaviour is modified.
Testing Instructions
No testing required - documentation-only update.
Deadline
None.