Skip to content

Update description of the Request / Has / Use pattern - #13

Open
RockYou-cmd wants to merge 1 commit into
SamMethot:mainfrom
RockYou-cmd:patch-1
Open

RockYou-cmd wants to merge 1 commit into
SamMethot:mainfrom
RockYou-cmd:patch-1

Conversation

@RockYou-cmd

@RockYou-cmd RockYou-cmd commented Sep 23, 2026 •

Copy link
Copy Markdown

Clarified the explanation of asset loading in GTA V.

Pull Request

What This Changes

A clear, one-paragraph description of what this PR does and why.

Type Of Change

  • Typo / broken link / formatting
  • Bug fix in an example
  • Outdated API reference
  • New lesson or section
  • Improved example (clearer / shorter / more correct)
  • Other (explain below)

Lesson(s) Affected

List the files you touched:

  • 01-basics/02-lua-crash-course.md
  • ...

Testing

If you changed a code example, confirm:

  • I tested the code on a dev server
  • The code matches the line-by-line annotation style of the rest of the repo
  • All ? placeholders are used in SQL (no string concat)
  • Any net event examples include source caching + arg validation
  • Doc links resolve

Tone Check

  • Plain-English summary at the top of any new lesson
  • Every non-trivial Lua line is annotated
  • TL;DR section at the bottom
  • Next: pointer at the end if it's a numbered lesson
  • No filler / marketing speak / unnecessary emoji
  • Doc links go to canonical sources (FiveM docs, coxdocs.dev, docs.qbox.re)

Updated Index?

  • Updated INDEX.md if I added or renamed files
  • N/A - didn't change file structure

Additional Notes

Anything reviewers should know.

Clarified the explanation of asset loading in GTA V.

This branch has not been deployed

No deployments
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.

1 participant