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

Add Inventory Order Management System #4237

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

go2ismail
Copy link

🔥A powerful and ready-to-use inventory management system, based on ASP.NET Core built with Monolithic Clean Architecture, CQRS, and MediatR. It features a completely RESTful API for seamless integration and a modern front-end powered by Vue.js.

General

✏️ Mark the necessary items without changing the structure of the PR template.

  • Pull request template structure not broken

Type

ℹ️ What types of changes does your code introduce?

👉 Put an x in the boxes that apply

  • Fix
  • Feature

Checklist

ℹ️ Check all checkboxes - this will indicate that you have done everything in accordance with the rules in CONTRIBUTING.

👉 Put an x in the boxes that apply.

  • Title as described
  • Make sure you put things in the right category!
  • Always add your items to the end of a list

Open Source

  • Link description does not contain a link to an author / third-party resource
  • The documentation (README) contains a description of the project, illustration of the project with a demo or screenshots and a CONTRIBUTING section
  • The documentation is in English.
  • The project is active and maintained.
  • The project accepts contributions.
  • Not a commercial product

Apps/Websites

  • The website is available without errors or ssl certificate problems, and load in a reasonable amount of time.
  • The website is using vuejs intensively. It should detect vue with vue-devtools.

    If you cannot detect vue with vue-devtools due to work at non public pages (e.g. for enterprise website), you can send Pull Request with screenshot that detected it.

  • The website is original and not too simple. For that reason, blogs and simple landing pages are rejected.
  • A commercial product using Vue, provided that guests could reasonably check out how Vue was used (i.e. A headless CMS which uses Vue for the Admin/editor Area and offers a free tier).

🔥A powerful and ready-to-use inventory management system, based on ASP.NET Core built with Monolithic Clean Architecture, CQRS, and MediatR. It features a completely RESTful API for seamless integration and a modern front-end powered by Vue.js.
@Jogai
Copy link
Collaborator

Jogai commented Feb 4, 2025

Where are the vue parts?

@go2ismail
Copy link
Author

Where are the vue parts?

All the pages (except index)
image

image

image

example:
image

@Jogai
Copy link
Collaborator

Jogai commented Feb 6, 2025

That does not seem like a best practice setup for vue. This is not a critique on your work, because this might be helpful to some people, its just that its not something to showcase to a broader audience.

@go2ismail
Copy link
Author

Thank you for the feedback. I believe it’s actually the opposite. For a multiple-page application where the HTML is server-side rendered (in this case by ASP.NET), Vue is extremely helpful due to its reactivity. It allows for a more seamless experience with less code. For example, I can use watch to observe changes without cluttering my code with event listeners just to update the DOM. All of this is achieved in a very simple manner, without requiring an additional build system, and the performance is already excellent—even without using Single File Components (SFC).

@Jogai
Copy link
Collaborator

Jogai commented Feb 7, 2025

Could be, but its specific to an environment where a server is serving pages. Since this is a vue list I think the audience is better off looking at vue specific solutions instead of depending on a whole different web framework.

@go2ismail
Copy link
Author

image
In my point of view, I have already met the required requirements. But in the end, I am aware that this is not my decision to make.

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