Skip to content

Add OnTrack Performance Tracking and Optimization Proposal document#46

Open
nouri-devv wants to merge 2 commits into
thoth-tech:mainfrom
nouri-devv:main
Open

Add OnTrack Performance Tracking and Optimization Proposal document#46
nouri-devv wants to merge 2 commits into
thoth-tech:mainfrom
nouri-devv:main

Conversation

@nouri-devv

Copy link
Copy Markdown

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

netlify Bot commented Aug 18, 2025

Copy link
Copy Markdown

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.

@ibi420 ibi420 left a comment

Copy link
Copy Markdown
Contributor

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

@returnMarcco returnMarcco left a comment

Copy link
Copy Markdown

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.

@WaelAlahamdi WaelAlahamdi left a comment

Copy link
Copy Markdown

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.

@MillicentAmolo MillicentAmolo left a comment

Copy link
Copy Markdown

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

@BrianDangDev BrianDangDev left a comment

Copy link
Copy Markdown
Contributor

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