p5.js-web-editor icon indicating copy to clipboard operation
p5.js-web-editor copied to clipboard

Removed redundant redirects between client and server

Open Mubashirshariq opened this issue 1 year ago • 1 comments

Fixes #2992

Changes: Server Side: Made changes on the server: Redirected "/sketches" and "/assets" to URLs with the username. Also, redirected "/reset-password" and "/reset-password/:reset_password_token" to "/account" if the user is logged in. client : Made changes on the client: Removed all Higher-Order Component (HOC) auth checks from routes.jsx, deleted createRedirectWithUsername component, and removed auth.js file.

I have verified that this pull request:

  • [ ] has no linting errors (npm run lint)
  • [ ] has no test errors (npm run test)
  • [ ] is from a uniquely-named feature branch and is up to date with the develop branch.
  • [ ] is descriptively named and links to an issue number, i.e. Fixes #123

Mubashirshariq avatar Feb 05 '24 07:02 Mubashirshariq

@lindapaiste plz review this PR ,i have made the proposed changes.

Mubashirshariq avatar Feb 05 '24 07:02 Mubashirshariq