Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Example to v2 list - Project Management System #768

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

IvanZemsky
Copy link

Background

Hi there, i've created a project for example list using FSD with react, typescript, mobx, material ui, react-hook-form.
Demo: https://ivanzemsky.github.io/ProjectManagement/
Code: https://github.com/IvanZemsky/ProjectManagement

I launched changed page locally, it seems to be working fine.

Changelog

  • src/pages/examples/_config.ts
  • src/pages/examples/img (added project-management-system.png)

Copy link

netlify bot commented Nov 25, 2024

👷 Deploy request for pr-fsd pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 835866a

@illright
Copy link
Member

Hi, thanks for the example! I briefly checked the project with Steiger, it's giving an error here — https://github.com/IvanZemsky/ProjectManagement/blob/master/src/entities/Executor/model/store.ts#L8

Looks like executor should be Executor. I wonder if this is a bug?

@IvanZemsky
Copy link
Author

IvanZemsky commented Nov 27, 2024

Looks like executor should be Executor. I wonder if this is a bug?

Huh, yeah, it seems like steiger didn't catch it. I've fixed the error.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants