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

feat: add login link to magic link form view #730

Merged
merged 2 commits into from
May 17, 2023
Merged

feat: add login link to magic link form view #730

merged 2 commits into from
May 17, 2023

Conversation

GeorgKott
Copy link
Contributor

@GeorgKott GeorgKott commented May 7, 2023

Supersedes #394
Closes #299

Thank you

@datamweb datamweb added the enhancement New feature or request label May 7, 2023
Copy link
Collaborator

@datamweb datamweb left a comment

Choose a reason for hiding this comment

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

@GeorgKott thank you for PR.
This issue has already been implemented in PR #394, only the test has not been added.

Can you do the same as PR #394 with the test?

And I edited your comment, why? you can read here.

@GeorgKott
Copy link
Contributor Author

I did the same as PR #394, exept changing "backToLogin" key to "usePassword". And add simple test.
Thank you for edited comment

@datamweb
Copy link
Collaborator

datamweb commented May 7, 2023

exept changing "backToLogin" key to "usePassword"

Yes, I mean using backToLogin. It seems to help the readability of the code. so please use backToLogin.

NOTE:
Make your commit messages meaningful.
Commit messages are important. They communicate the intent of a specific change, concisely. They make it easier to review code, and to find out why a change was made if the code history is examined later.

The audience for your commit messages will be the codebase maintainers, any code reviewers, and debuggers trying to figure out when a bug might have been introduced.

There are some references for writing good commit messages:
Git Best Practices — AFTER Technique - DZone DevOps
Semantic Commit Messages
Conventional Commits

more info see here.

@GeorgKott
Copy link
Contributor Author

exept changing "backToLogin" key to "usePassword"

Yes, I mean using backToLogin. It seems to help the readability of the code. so please use backToLogin.

NOTE: Make your commit messages meaningful. Commit messages are important. They communicate the intent of a specific change, concisely. They make it easier to review code, and to find out why a change was made if the code history is examined later.

The audience for your commit messages will be the codebase maintainers, any code reviewers, and debuggers trying to figure out when a bug might have been introduced.

There are some references for writing good commit messages: Git Best Practices — AFTER Technique - DZone DevOps Semantic Commit Messages Conventional Commits

more info see here.

Thank you for links! I did some changes

Copy link
Collaborator

@datamweb datamweb left a comment

Choose a reason for hiding this comment

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

@GeorgKott thank you!
LGTM.

@kenjis kenjis changed the title add login link to magic link form view feat: add login link to magic link form view May 17, 2023
@kenjis kenjis merged commit 2a3d67c into codeigniter4:develop May 17, 2023
@kenjis
Copy link
Member

kenjis commented May 17, 2023

@GeorgKott Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Missing back link on Magic Link form
3 participants