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.

Sign up for GitHub

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

Jump to bottom

Passwordless login for your PHP project #1054

Open
wants to merge 31 commits into
base: develop
Choose a base branch
from
Open

Passwordless login for your PHP project #1054

wants to merge 31 commits into from

Conversation

oyedeletemitope
Copy link

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

  • You created a branch from dev branch.
  • You PR is raised on dev branch and not onmaster.
  • Blog follows Blog Guidelines.
  • You have read the Contribution Guidelines before creating this PR.
  • Checked for any duplicate blog entry.
  • Checked for all grammatical errors in the blog.
  • You have not published the blog content anywhere else on the internet on platforms like medium, dev.to etc.
  • The content is not copy-pasted from anywhere else on the Internet.
Description of change

pareek-naren and others added 28 commits December 15, 2021 14:53
@pareek-naren
Merge pull request LoginRadius#923 from LoginRadius/dev
2d68342 
Merge Dev into Master, improved build time
@raghunath-r-a
Updated redirects for a couple of URLs
28758d6
@pareek-naren
Merge pull request LoginRadius#929 from raghunath-r-a/dev
8c12932 
Updated redirects for a couple of URLs
@pareek-naren
Merge pull request LoginRadius#930 from LoginRadius/dev
658479f 
Dev to Master: A few content updates and URL redirects
@espark-biz @raghunath-r-a
Blog - securing php api with jwt (LoginRadius#724)
fcae450 
* title : added new blog of php api using jwt

* added heading tags and change some code

* added jwt demo application link

* Made content improvements

* added interlinking

* Improved and proofread content; added internal linking

Co-authored-by: Viral Prajapati <viral.prajapati@esparkbizmail.com>
Co-authored-by: Raghunath Reddy <11178736+raghunath-r-a@users.noreply.github.com>
@GauravRoy6 @pareek-naren @raghunath-r-a
New Blog - React security vulnerabilities and other changes (LoginRad…
1ad3929 
…ius#927)


Co-authored-by: Narendra Pareek <narendra.rtgh@gmail.com>
Co-authored-by: Raghunath Reddy <11178736+raghunath-r-a@users.noreply.github.com>
@raghunath-r-a
1. deleted a redundant/useless folder
283b1a2 
2. modified dates for two blogs to be pushed into production
3. Fulfilled a linking request
@raghunath-r-a
Merge pull request LoginRadius#940 from LoginRadius/dev
c2f7584 
Dev to Master - 2 new blog posts and some changes
@mohammed786
Removed unwanted dependecies (LoginRadius#943)
268f235
@FuzzySid @fuzzysid99 @raghunath-r-a
Blog Local Storage vs Session Storage vs Cookies added (LoginRadius#942)
28d5878 
* Blog Local Storage vs Session Storage vs Cookies added

* Improved and proofread content

* minor changes

Co-authored-by: fuzzysid99 <siddhant@quinn.care>
Co-authored-by: Raghunath Reddy <11178736+raghunath-r-a@users.noreply.github.com>
@raghunath-r-a
minor fixes and updated date
0ec40da
@mohammed786
Merge pull request LoginRadius#953 from LoginRadius/dev
55c2c4d 
Dev to Master: A new blog and other changes
@nefejames @raghunath-r-a
Blog - Next.js Authentication (LoginRadius#928)
9bc3295 
* title: adding Next.js Authentication article

* Improved content; proofread


Co-authored-by: Raghunath Reddy <11178736+raghunath-r-a@users.noreply.github.com>
@raghunath-r-a
Merge pull request LoginRadius#956 from LoginRadius/dev
30ac5ab 
Dev to Master: New blog on Next.js Authentication
@uma-victor1 @raghunath-r-a
An article on nodejs user authentication (LoginRadius#950)
fb7222f 
* added changes from PR LoginRadius#941

* improvements; proofread

Co-authored-by: Raghunath Reddy <11178736+raghunath-r-a@users.noreply.github.com>
@GauravRoy6
Updated another Blog with grammatical fixes and formatting changes (L…
0339f7b 
…oginRadius#947)
@mohammed786
Merge pull request LoginRadius#959 from LoginRadius/dev
10fc0d0 
Dev to Master: An article on nodejs user authentication
@raghunath-r-a @IArchna
New blog on bigcommerce (LoginRadius#966)
4ab9190 
Co-authored-by: IArchna <71074509+IArchna@users.noreply.github.com>
@raghunath-r-a
minor changes
97f5baa
@mohammed786
Merge pull request LoginRadius#969 from LoginRadius/dev
df7c6b4 
New blog on bigcommerce (LoginRadius#966)
@oyedeletemitope
my first article
0b711bf
@raghunath-r-a
rebase with new blog structure
eec8ada
@raghunath-r-a
initial checks
3e976f2
@raghunath-r-a
initial checks - 2
3737b5c
@oyedeletemitope
changes
9f61b33
@oyedeletemitope
changes
baaff8c
@oyedeletemitope
changes
efca745
@oyedeletemitope
updated
dfe6d32
@oyedeletemitope
Copy link
Author

Hello @raghunath-r-a , here is the new pr for the article

@raghunath-r-a
Copy link
Collaborator

@oyedeletemitope

  1. The PR shall be raised against the develop branch
  2. It seems it has some serious issues as the PR is showing too many changes.

Please consider fixing them to help us proceed further.

@oyedeletemitope
Copy link
Author

@raghunath-r-a please what do I need to do to fix them

@oyedeletemitope oyedeletemitope changed the base branch from master to develop April 21, 2022 08:11
@oyedeletemitope
Copy link
Author

Does this resolve it

@oyedeletemitope
Copy link
Author

Hello @raghunath-r-a I made some changes earlier as requested. Just wanted to know if it’s now sorted

@raghunath-r-a
Copy link
Collaborator

@oyedeletemitope We'll update you on this in the next few working days

@raghunath-r-a
Copy link
Collaborator

Hi @oyedeletemitope

I have fixed the issues again.

Please delete your local master branch: git branch -D master

Then checkout the master branch fully: git checkout --track origin/master

Then make your changes and commit them. Then push them to GitHub.

@oyedeletemitope
Copy link
Author

done as requested I hope it works this time

@oyedeletemitope
Copy link
Author

Hello @raghunath-r-a did it work out?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
9 participants
@oyedeletemitope @raghunath-r-a @pareek-naren @espark-biz @GauravRoy6 @mohammed786 @FuzzySid @nefejames @uma-victor1