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

[Security] make:security:custom #19809

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

Conversation

jrushlow
Copy link
Contributor

@jrushlow jrushlow commented Apr 21, 2024

Since MakerBundle v1.59.0 - you can call make:security:custom to generate a simple, no-frills, custom authenticator based off the example in the docs.

Since MakerBundle `v1.59.0` - you can call `make:security:custom` to generate a simple, no-frills, custom authenticator based off the example in the docs.
@carsonbot carsonbot added this to the 6.4 milestone Apr 21, 2024
@carsonbot carsonbot changed the title [security] make:security:custom [Security] make:security:custom Apr 21, 2024
Comment on lines +8 to +9
cases, you can use the ``make:security:custom`` command to create your own
authenticator.
Copy link
Contributor Author

@jrushlow jrushlow Apr 21, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm.. we reference the make:* command in the doctrine section like I have here.. But we use the code-block "syntax" else where in the doc's..

.. code-block:: terminal
I'm thinking we should make this a code block here?

Sidenote - I should probably change the

You can use the ``make:entity`` command to create this class and any fields you
reference to a "code-block" style as well for consistency. Thoughts?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I think the terminal code block would be better because it stands out a lot and it's easier to scan fore readers. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants