Vorleak Yek

Results 11 comments of Vorleak Yek

@jng34, maybe it's good to create a PR for this so that I can take a look at your code because I plan to work on [#1754](https://github.com/hackforla/VRMS/issues/1754), which is related...

@trillium, can you confirm that the accessLevel in the dummy data object should be "admin" or "projectLead"? ``` const user = { _id: 1, name: { firstName: "John", lastName: "Doe"...

![image](https://github.com/user-attachments/assets/10cf7cb4-2742-4dc7-afd9-9b0f4f6c1c17) @trillium, can you confirm this is okay, or should this PR include the changes in #1798? It should link to "/users". But, for the purpose of testing without the...

@ntrehan, if possible can you provide some info on how to test this in the backend? Thanks!

@ntrehan, thanks for the details testing information. It's very helpful! The code looks good, and it seems like the return data have one "superadmin" and the rest are "admin" for...

@JackHaeg, @trillium, the search result in this ticket should be based on the dummy data created in https://github.com/hackforla/VRMS/issues/1752. Is that correct? It looks like we're doing the API integration in...

@JackHaeg , @trillium, are these last two checkboxes for information only? It seems like we will be covering them in https://github.com/hackforla/VRMS/issues/1752. ![image](https://github.com/user-attachments/assets/6c1bdf7c-881e-4e7b-90bc-deb7a4d46f4d)

@trillium, @chukalicious, should we add style to fit the header in one line as before? The old style looks better than the new one. Also, we don't need to convert...

@pluto-bell, thanks for taking a look at the PR. I think you're right that the package.json, package-lock.json and yarn.lock files shouldn't be included in the PR. @trillium, is this okay?...

Thanks @trillium for helping me restore the JSON files yesterday. I also updated the code in `client/src/pages/Users.jsx` to use "Box" and "Container" MUI components instead of the "div" element, and...