Skip to content

Conversation

@nouri-devv
Copy link

Description

Type of change

  • [ x] New feature (non-breaking change which adds functionality) - new proposal

How Has This Been Tested?

Testing Checklist

  • [ x] Tested in latest Chrome
  • [ x] Tested in latest Safari
  • [x ] Tested in latest Firefox

Checklist

  • [ x] My code follows the style guidelines of this project
  • [ x] I have performed a self-review of my own code
  • [ x] I have commented my code in hard-to-understand areas
  • [ x] I have made corresponding changes to the documentation
  • [ x] My changes generate no new warnings
  • [ x] I have requested a review from ... on the Pull Request

@netlify
Copy link

netlify bot commented Aug 18, 2025

Deploy Preview for ontrackdocumentation ready!

Name Link
🔨 Latest commit eec05af
🔍 Latest deploy log https://app.netlify.com/projects/ontrackdocumentation/deploys/68e39ac2bc622600084735db
😎 Deploy Preview https://deploy-preview-46--ontrackdocumentation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Contributor

@ibi420 ibi420 left a comment

Choose a reason for hiding this comment

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

Hello @nouri-devv,

This proposal clearly highlights a practical way to improve OnTrack’s performance and accessibility. It’s direct, well-written, and provides actionable recommendations on how to begin. My only feedback is on naming conventions: please avoid spaces in file and directory names, and instead use either hyphens or underscores.

Thank you for the opportunity to review this and good job

Copy link

@returnMarcco returnMarcco left a comment

Choose a reason for hiding this comment

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

Good job @nouri-devv . The documentation clearly states the proposal, the reasoning behind it, and the plan. It is well formatted and reads well.

Copy link

@WaelAlahamdi WaelAlahamdi left a comment

Choose a reason for hiding this comment

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

Hi @nouri-devv, I have reviewed the proposal. The document is clear, well-structured, and provides practical recommendations for performance tracking and optimization. Thank you for the work, approved.

Copy link

@MillicentAmolo MillicentAmolo left a comment

Choose a reason for hiding this comment

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

The proposal is clear and well structured, good job

Copy link
Contributor

@BrianDangDev BrianDangDev left a comment

Choose a reason for hiding this comment

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

Missing title is causing an error add the title, then I’ll approve the PR.

@nouri-devv nouri-devv reopened this Oct 6, 2025
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.

6 participants