feat(templates): ecommerce template v3 #8297
Conversation
|
|
||
| export const adminsOrPublished: Access = ({ req: { user } }) => { | ||
| if (user) { | ||
| return checkRole(['admin'], user) |
There was a problem hiding this comment.
I know this is a draft PR, but shouldn't this be like this:
if (user) {
if (checkRole(['admin'], user)) {
return true
}
}If you are logged in, but not an admin, we still need to check if the page status is published or not.
The current version will return false for a logged in normal user even if the page status is marked as published
|
I've just been trying to get it to ~work~ with the latest beta. I made a lot of changes- some type related, some updating to latest api. I started with the payload-vercel-postgres-template and then once I got it to work I moved over to the feat/ecommerce-template branch and tried to merge it in. Not sure if this is even a help at all, but though I would just put it here in case it is. https://github.com/jcourson8/payload/tree/feat/ecommerce-3-update |
No description provided.