Skip to content

docs: fix incorrect sample code in restful.rst#8253

Merged
kenjis merged 2 commits intocodeigniter4:developfrom
kenjis:docs-restful-controller-ns
Nov 27, 2023
Merged

docs: fix incorrect sample code in restful.rst#8253
kenjis merged 2 commits intocodeigniter4:developfrom
kenjis:docs-restful-controller-ns

Conversation

@kenjis
Copy link
Copy Markdown
Member

@kenjis kenjis commented Nov 25, 2023

Description
Fixes #8248

Checklist:

  • Securely signed commits
  • [] Component(s) with PHPDoc blocks, only if necessary or adds value
  • [] Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@sammyskills
Copy link
Copy Markdown
Contributor

I think it would be nice to also show the example using array callable syntax, where a namespace is specified or backward slash is appended, just like you have in the #8248 (comment)

@kenjis
Copy link
Copy Markdown
Member Author

kenjis commented Nov 25, 2023

Added sample code:
Screenshot 2023-11-25 21 11 34

@sammyskills
Copy link
Copy Markdown
Contributor

Thank you @kenjis

@kenjis kenjis merged commit 35a8f74 into codeigniter4:develop Nov 27, 2023
@kenjis kenjis deleted the docs-restful-controller-ns branch November 27, 2023 23:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Pull requests for documentation only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: Resource and Presenter routes does not accept qualified names for controller option

3 participants