diff --git a/jam-ui/cypress/e2e/auth/forgot-password.cy.js b/jam-ui/cypress/e2e/auth/forgot-password.cy.js index 4997a8870..ea3b02ffb 100644 --- a/jam-ui/cypress/e2e/auth/forgot-password.cy.js +++ b/jam-ui/cypress/e2e/auth/forgot-password.cy.js @@ -12,17 +12,17 @@ describe('forgot password', () => { it('redirects to forgot password page', () => { cy.visit('/'); - cy.url().should('include', '/authentication/basic/login'); + cy.url().should('include', '/auth/login'); cy.get('a') .contains('Forgot password?') .click(); - cy.url().should('include', '/authentication/basic/forget-password'); + cy.url().should('include', '/auth/forget-password'); cy.get('h5').contains('Forgot Your Password'); }); describe('validate forgot password form', () => { beforeEach(() => { - cy.visit('/authentication/basic/forget-password'); + cy.visit('/auth/forget-password'); cy.get('[data-testid=email]').clear(); cy.get('[data-testid=submit]').should('be.disabled'); }); @@ -31,14 +31,14 @@ describe('forgot password', () => { it('invalid email format', () => { cy.get('[data-testid=email]').type('invalid-email-format@example'); cy.get('[data-testid=submit]').click(); - cy.url().should('not.include', /\/authentication\/basic\/confirm-mail?\S+/); + cy.url().should('not.include', /\/auth\/confirm-mail?\S+/); }); //valid email format but non-existing it('valid email format but non-existing', () => { cy.get('[data-testid=email]').type('valid-email-format@example.com'); cy.get('[data-testid=submit]').click(); - cy.url().should('not.include', /\/authentication\/basic\/confirm-mail?\S+/); + cy.url().should('not.include', /\/auth\/confirm-mail?\S+/); }); //valid and existing email @@ -47,7 +47,7 @@ describe('forgot password', () => { cy.get('[data-testid=submit]').click(); cy.wait(3000); cy.contains('Please check your email!'); - cy.url().should('match', /\S+authentication\/basic\/confirm-mail?\S+/); + cy.url().should('match', /\S+auth\/confirm-mail?\S+/); cy.contains('An email has been sent to nuwan@jamkazam.com.') }); }); diff --git a/jam-ui/cypress/e2e/auth/login.cy.js b/jam-ui/cypress/e2e/auth/login.cy.js index 7cf1808f1..917007332 100644 --- a/jam-ui/cypress/e2e/auth/login.cy.js +++ b/jam-ui/cypress/e2e/auth/login.cy.js @@ -31,7 +31,7 @@ describe('Unauthenticated users redirect to login page', () => { it('redirects to login page', () => { cy.clearCookie('remeber_token') cy.visit('/') - cy.url().should('include', '/authentication/basic/login') + cy.url().should('include', '/auth/login') cy.contains('Sign In') }) }) @@ -42,7 +42,7 @@ describe('Login page', () => { }) it('validate login form', () => { - cy.visit('/authentication/basic/login') + cy.visit('/auth/login') cy.reload() cy.get('[data-testid=submit]').should('be.disabled') cy.get('[data-testid=email]').type('invalid-email-format@example') @@ -54,7 +54,7 @@ describe('Login page', () => { }) it('submit login form with invalid credentials', () => { - cy.visit('/authentication/basic/login') + cy.visit('/auth/login') cy.reload() cy.get('[data-testid=email]').type('peter@example.com') cy.get('[data-testid=password]').type('wrong') @@ -63,7 +63,7 @@ describe('Login page', () => { }) it('submits login form', () => { - cy.visit('/authentication/basic/login') + cy.visit('/auth/login') submitLogin() cy.url().should('eq', Cypress.config().baseUrl + '/') // tests won't fail in case the port changes //cy.contains('Signed in as peter@example.com') @@ -73,7 +73,7 @@ describe('Login page', () => { it('redirect to requested page', () => { cy.visit('/friends') - cy.url().should('include', '/authentication/basic/login') + cy.url().should('include', '/auth/login') cy.reload() submitLogin() cy.url().should('eq', Cypress.config().baseUrl + '/friends') @@ -83,7 +83,7 @@ describe('Login page', () => { describe('Forget password page', () => { it('submit forget password form', () => { - cy.visit('/authentication/basic/forget-password') + cy.visit('/auth/forget-password') cy.get('[data-testid=email]').type('peter@example.com') cy.get('[data-testid=submit]').click() cy.contains('An email is sent') diff --git a/jam-ui/cypress/e2e/layout/navigation.cy.js b/jam-ui/cypress/e2e/layout/navigation.cy.js index 62a6aa5d8..441dd06ff 100644 --- a/jam-ui/cypress/e2e/layout/navigation.cy.js +++ b/jam-ui/cypress/e2e/layout/navigation.cy.js @@ -25,7 +25,7 @@ describe('Top Navigation', () => { it('not allowed to protected page', () => { cy.visit('/friends'); cy.wait('@getAppFeatures'); - cy.url().should('include', '/authentication/basic/login'); + cy.url().should('include', '/auth/login'); cy.contains('Sign in'); cy.get('button').should('have.text', 'Sign in'); cy.get('[data-testid=navbarTopProfileDropdown]').should('not.exist'); diff --git a/jam-ui/src/components/auth/ConfirmMailContent.js b/jam-ui/src/components/auth/ConfirmMailContent.js index a1a05a3ef..1010e358c 100644 --- a/jam-ui/src/components/auth/ConfirmMailContent.js +++ b/jam-ui/src/components/auth/ConfirmMailContent.js @@ -21,7 +21,7 @@ const ConfirmMailContent = ({ layout, titleTag: TitleTag }) => {

{t('confirmEmailContent.description_3')}

- diff --git a/jam-ui/src/components/auth/ForgetPasswordForm.js b/jam-ui/src/components/auth/ForgetPasswordForm.js index 97c82379e..cdd3458fc 100644 --- a/jam-ui/src/components/auth/ForgetPasswordForm.js +++ b/jam-ui/src/components/auth/ForgetPasswordForm.js @@ -32,7 +32,7 @@ const ForgetPasswordForm = ({ setRedirect, setRedirectUrl, layout }) => { }; useEffect(() => { - setRedirectUrl(`/authentication/${layout}/confirm-mail?email=${email}`); + setRedirectUrl(`/auth/confirm-mail?email=${email}`); }, [setRedirectUrl, layout, email]); return ( diff --git a/jam-ui/src/components/auth/LoginForm.js b/jam-ui/src/components/auth/LoginForm.js index f4c929914..2c656ddc7 100644 --- a/jam-ui/src/components/auth/LoginForm.js +++ b/jam-ui/src/components/auth/LoginForm.js @@ -83,7 +83,7 @@ const LoginForm = ({ setRedirect, hasLabel, layout }) => { /> - + Forget Password? {/* Forgot Password */} diff --git a/jam-ui/src/components/auth/PasswordResetForm.js b/jam-ui/src/components/auth/PasswordResetForm.js index 494fc624c..2c2cd15ad 100644 --- a/jam-ui/src/components/auth/PasswordResetForm.js +++ b/jam-ui/src/components/auth/PasswordResetForm.js @@ -45,7 +45,7 @@ const PasswordResetForm = ({ setRedirect, setRedirectUrl, layout, hasLabel }) => }; useEffect(() => { - setRedirectUrl(`/authentication/${layout}/login`); + setRedirectUrl(`/auth/login`); }, [setRedirectUrl, layout]); // useEffect(() => { diff --git a/jam-ui/src/components/auth/RegistrationForm.js b/jam-ui/src/components/auth/RegistrationForm.js index 70b00598d..50bb7cdb6 100644 --- a/jam-ui/src/components/auth/RegistrationForm.js +++ b/jam-ui/src/components/auth/RegistrationForm.js @@ -24,7 +24,7 @@ const RegistrationForm = ({ setRedirect, setRedirectUrl, layout, hasLabel }) => }; useEffect(() => { - setRedirectUrl(`/authentication/${layout}/login`); + setRedirectUrl(`/auth/login`); }, [setRedirectUrl, layout]); useEffect(() => { diff --git a/jam-ui/src/components/auth/basic/JKRegistration.js b/jam-ui/src/components/auth/basic/JKRegistration.js index d0e5dc43c..a54e03a8c 100644 --- a/jam-ui/src/components/auth/basic/JKRegistration.js +++ b/jam-ui/src/components/auth/basic/JKRegistration.js @@ -77,7 +77,7 @@ const Registration = () => {

- {t('signupForm.alreadyHaveAccount')} {t('signin')} + {t('signupForm.alreadyHaveAccount')} {t('signin')}

diff --git a/jam-ui/src/components/auth/basic/Login.js b/jam-ui/src/components/auth/basic/Login.js index 74aba8877..c813790b4 100644 --- a/jam-ui/src/components/auth/basic/Login.js +++ b/jam-ui/src/components/auth/basic/Login.js @@ -15,7 +15,7 @@ const Login = () => {

- or {t('signup')} + or {t('signup')} {/* {t('signup')} */}

diff --git a/jam-ui/src/components/auth/basic/Registration.js b/jam-ui/src/components/auth/basic/Registration.js index 52c4fc924..ac236152b 100644 --- a/jam-ui/src/components/auth/basic/Registration.js +++ b/jam-ui/src/components/auth/basic/Registration.js @@ -11,7 +11,7 @@ const Registration = () => (

- Have an account? Login + Have an account? Login

diff --git a/jam-ui/src/components/jamtracks/JKJamTracksLandingBody.js b/jam-ui/src/components/jamtracks/JKJamTracksLandingBody.js index 7d4b23d86..2697833bd 100644 --- a/jam-ui/src/components/jamtracks/JKJamTracksLandingBody.js +++ b/jam-ui/src/components/jamtracks/JKJamTracksLandingBody.js @@ -60,7 +60,7 @@ const BodyComponent = ({ } else { console.log("no user") // redirect to the signup screen with the jamtrack & artist params set - window.location.href = "/authentication/basic/signup?artist=" + encodeURIComponent(artist) + '&jamtrack=' + encodeURIComponent(slug) + window.location.href = "/auth/signup?artist=" + encodeURIComponent(artist) + '&jamtrack=' + encodeURIComponent(slug) } } else { diff --git a/jam-ui/src/components/navbar/JKTopNavRightSideNavItem.js b/jam-ui/src/components/navbar/JKTopNavRightSideNavItem.js index 7684cb968..9007db251 100644 --- a/jam-ui/src/components/navbar/JKTopNavRightSideNavItem.js +++ b/jam-ui/src/components/navbar/JKTopNavRightSideNavItem.js @@ -43,7 +43,7 @@ const TopNavRightSideNavItem = () => { ) : ( - + {t('signin', { ns: 'auth' })} diff --git a/jam-ui/src/components/navbar/ProfileDropdown.js b/jam-ui/src/components/navbar/ProfileDropdown.js index ca0eaa212..491d691c1 100644 --- a/jam-ui/src/components/navbar/ProfileDropdown.js +++ b/jam-ui/src/components/navbar/ProfileDropdown.js @@ -42,7 +42,7 @@ const ProfileDropdown = () => { Settings - + Logout diff --git a/jam-ui/src/helpers/privateRoute.js b/jam-ui/src/helpers/privateRoute.js index 786fa78b9..d156054b3 100644 --- a/jam-ui/src/helpers/privateRoute.js +++ b/jam-ui/src/helpers/privateRoute.js @@ -19,7 +19,7 @@ const PrivateRoute = ({ component: Component, ...otherProps }) => { // diff --git a/jam-ui/src/layouts/JKLayout.js b/jam-ui/src/layouts/JKLayout.js index dd3f33db9..cb216c029 100644 --- a/jam-ui/src/layouts/JKLayout.js +++ b/jam-ui/src/layouts/JKLayout.js @@ -21,7 +21,7 @@ const Layout = () => { }> - + diff --git a/jam-ui/src/layouts/Layout.js b/jam-ui/src/layouts/Layout.js index 32234a223..017814ec1 100644 --- a/jam-ui/src/layouts/Layout.js +++ b/jam-ui/src/layouts/Layout.js @@ -30,7 +30,7 @@ const Layout = () => { - + } position={toast.POSITION.BOTTOM_LEFT} /> diff --git a/jam-ui/src/routes.js b/jam-ui/src/routes.js index b7bc08b81..3907dc885 100644 --- a/jam-ui/src/routes.js +++ b/jam-ui/src/routes.js @@ -129,20 +129,20 @@ export const homeRoutes = { export const authenticationRoutes = { name: 'Authentication', - to: '/authentication', + to: '/auth', icon: 'lock', children: [ { - to: '/authentication/basic', + to: '/auth/basic', name: 'Basic', children: [ - { to: '/authentication/basic/login', name: 'Login' }, - { to: '/authentication/basic/logout', name: 'Logout' }, - { to: '/authentication/basic/register', name: 'Register' }, - { to: '/authentication/basic/forget-password', name: 'Forgot password' }, - { to: '/authentication/basic/password-reset', name: 'Reset password' }, - { to: '/authentication/basic/confirm-mail', name: 'Confirm mail' }, - { to: '/authentication/basic/lock-screen', name: 'Lock screen' } + { to: '/auth/login', name: 'Login' }, + { to: '/auth/logout', name: 'Logout' }, + { to: '/auth/register', name: 'Register' }, + { to: '/auth/forget-password', name: 'Forgot password' }, + { to: '/auth/password-reset', name: 'Reset password' }, + { to: '/auth/confirm-mail', name: 'Confirm mail' }, + { to: '/auth/lock-screen', name: 'Lock screen' } ] }, { diff --git a/ruby/lib/jam_ruby/models/user.rb b/ruby/lib/jam_ruby/models/user.rb index f3e1c6855..491cfedec 100644 --- a/ruby/lib/jam_ruby/models/user.rb +++ b/ruby/lib/jam_ruby/models/user.rb @@ -1126,7 +1126,7 @@ module JamRuby user.reset_password_token_created = Time.now user.save(validate:false) - reset_url = "#{base_uri}/authentication/basic/reset_password_token?token=#{user.reset_password_token}&email=#{CGI.escape(email)}" + reset_url = "#{base_uri}/auth/reset_password_token?token=#{user.reset_password_token}&email=#{CGI.escape(email)}" UserMailer.password_reset(user, reset_url).deliver_now user end