Skip to content
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

Update radio.blade.php #5502

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

vonsofh
Copy link
Contributor

@vonsofh vonsofh commented Apr 16, 2024

fix my problems, yeah as simple as that...

WHY

Because, when my radio contains backslash () it can't be checked by previous code

BEFORE - What was wrong? What was happening before this PR?

It doesn't checked the radio because value contains backslash ()

AFTER - What is happening after this PR?

Now it can, because using filter

HOW

How did you achieve that, in technical terms?

I tried to make morph relationship, then when I put value App\Models\Room it can't checked the radio when I tried to edit the entry

Is it a breaking change?

I think it is not, just a small fix

How can we test the before & after?

I tested and it was working both for integer value or with backslash ()

fix my problems
@karandatwani92 karandatwani92 added Possible Bug A bug that was reported but not confirmed yet. Size: XS 1 hour labels Apr 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Possible Bug A bug that was reported but not confirmed yet. Size: XS 1 hour
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants