forked from laravel/docs
-
Notifications
You must be signed in to change notification settings - Fork 81
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
合併 laravel/docs 內容 #478
Merged
Merged
合併 laravel/docs 內容 #478
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Update http-tests.md * Update http-tests.md * Update http-tests.md Co-authored-by: Taylor Otwell <[email protected]>
… default() modifier (#7938) * Document the usage of DB::raw() within the default value as to not be used as this could cause issues with doctrine/dbal. * Update migrations.md Co-authored-by: Taylor Otwell <[email protected]>
* Update billing.md * Update cashier-paddle.md * Update billing.md * Update billing.md * Update billing.md * Update cashier-paddle.md * Update billing.md * Update billing.md * Update billing.md Co-authored-by: Taylor Otwell <[email protected]>
* Add orWhereRelation and orWhereMorphRelation * Update eloquent-relationships.md Co-authored-by: Taylor Otwell <[email protected]>
* Updating custom throttle reponse. Added the 2 parameters that the custom response callback is called with in the ThrottleRequests::buildException method. I tried to implement my own custom response and lost the throttle headers (X-RateLimit-Limit, etc). It was not obvious to me how to get the header info into my response. I tried to dig through the code, but missed the spot where the callback is called. Got a response to my problem on Laracasts, through I'd help put it in the docs for others. * Update routing.md Co-authored-by: Taylor Otwell <[email protected]>
* Adding Declaring Model Engine to Scout docs * Matching styling of docs by pulling in EngineManager namespace as import * formatting Co-authored-by: Taylor Otwell <[email protected]>
* Update validation.md * Update validation.md Co-authored-by: Taylor Otwell <[email protected]>
* Adds documentation for Str::isJson() * Update helpers.md Co-authored-by: Taylor Otwell <[email protected]>
* Adds documentation for Arr::join * Updates link * Formatting Co-authored-by: Taylor Otwell <[email protected]>
* add validation response example * formatting Co-authored-by: Taylor Otwell <[email protected]>
* Cashier v14 changes * Update billing.md * Update billing.md Co-authored-by: Taylor Otwell <[email protected]>
* Update dusk.md * Update dusk.md * Update dusk.md Co-authored-by: Taylor Otwell <[email protected]>
… connection (#8154) * Updates queues.md - Add explicit documentation about `sync` to 'Dispatching Jobs' section Adds more explicit documentation to the 'Dispatching Jobs' section regarding Laravel's default queue connection of `sync`. I am using queues in Laravel for the first time, and this was a footgun for me. As I was trying to dispatch jobs to my queue, I noticed that they were running synchronously and I immediately assumed I was doing something wrong, even though I followed the docs. I would expect this to be documented a little more explicitly which would have helped with my development experience. * Update queues.md * Update queues.md * Update queues.md Co-authored-by: Taylor Otwell <[email protected]>
…8159) * The isAssoc variable name is misleading. I think it was accidently copied from the previous help topic. Changing to isList * removed accidently added brackets
* [9.x] Update implicit rules description to a new invokable way * Update validation.md Co-authored-by: Taylor Otwell <[email protected]>
Removes "the" before "Fortify's" in fortify.md
* Return installation using `laravel new` to docs * Update installation.md Co-authored-by: Taylor Otwell <[email protected]>
… improve search/find in page (#8174) Co-authored-by: Simon Minton <[email protected]>
The current docs have two things that need to be adjusted to give better clarity into the `decimal` cast. First, there is a space between "decimal:" and "<digits>" that is created because of putting two code blocks next to each other. It could easily be conceived that the cast should be written as `decimal: 2`, for example with a space between the colon and the precision value. This PR corrects that by wrapping the entire cast example in a single code block. Secondly, the "digits" parameter has been changed to be "precision" as that gives better understanding as to what that value is intended to do/be. While the `asDecimal` method in `HasAttributes` and the `number_format` function call this parameter "decimals", both "decimals" and "digits" are vague as to their intended purpose in the usage of this cast. Let me know if you need me to adjust or change anything or you have any questions. Thanks!
* Update http-tests.md * Update http-tests.md Co-authored-by: Taylor Otwell <[email protected]>
Insted of selected="selected", that sounds very old and to prevent horizzontal scroll on desktop
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.