BioDrop icon indicating copy to clipboard operation
BioDrop copied to clipboard

move premium check to middleware

Open dmitrykulakovfrontend opened this issue 1 year ago • 4 comments

Fixes Issue

Closes #9630

Changes proposed

Update JWT, refactor middleware.js, remove premium checks in getServerSideProps and move them to middleware.js

Check List (Check all the applicable boxes)

  • [x] My code follows the code style of this project.
  • [ ] My change requires changes to the documentation.
  • [ ] I have updated the documentation accordingly.
  • [x] All new and existing tests passed.
  • [x] This PR does not contain plagiarized content.
  • [x] The title of my pull request is a short description of the requested changes.

dmitrykulakovfrontend avatar Nov 07 '23 19:11 dmitrykulakovfrontend

Thank you for making the changes. Could you check your Prettier settings, it looks like extra lines are being reformatted, which makes it more difficult to review "actual" changes

eddiejaoude avatar Nov 15 '23 09:11 eddiejaoude

I'm apologizing for the issue with prettier, it should be fixed now

dmitrykulakovfrontend avatar Nov 22 '23 06:11 dmitrykulakovfrontend

I'm apologizing for the issue with prettier, it should be fixed now

thank you 👍 sorry for my slow reply, I will be looking at this PR next

eddiejaoude avatar Nov 28 '23 07:11 eddiejaoude

I can fix the conflicts, as it has conflicted with my changes

eddiejaoude avatar Dec 08 '23 05:12 eddiejaoude