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

created a new frontend application using Typescript Nuxt3 Vue3 and pi… #392

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

Conversation

Zaker237
Copy link
Member

@Zaker237 Zaker237 commented Apr 7, 2023

This pull request create a new frontend application

Why ?
as the application grows it is important that it is more robust. the use of typescript facilitates this and also reduces runtime errors

How ?
i generated a new app using Nuxt3 and rewrite the App using the same components and pages

Steps to verify:
follows the same step as the steps of old app

@elhmn
Copy link
Member

elhmn commented Apr 10, 2023

This is a massive PR, out of curiosity, why are we doing this rewrite in the middle of the current implementation ?

Copy link
Member

@elhmn elhmn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it is good if we understand why this is needed and if it can't wait for the current work to be done, so I left a question on the PR to clarify that

@Zaker237
Copy link
Member Author

it's indeed a mistake to have tagged you on this, it should have waited and it currently has no priority. I was learning how the new stack works and built the project with it straight away as we had discussed it once.

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