BuildForSDG / Phenom-Frontend

Team-056 frontent solution for BuildForSDG 2020 Innovation challenge.
MIT License
0 stars 5 forks source link

Framework: React for building our frontend solution #2

Closed onyijne closed 4 years ago

onyijne commented 4 years ago

Current template cannot serve our need . As seen at js-starter, the base template for our frontend solution is relatively without code we could work on based on the technical requirements of our project.

A tested and trusted framework will be solid ground to start on React being a modern framework for developing robust and enterprise ready frontend application is a good option based on our frontend innovator's experiences.

Merging both templates The current js-starter template has a basic standard
which must be kept if we want good code ratings, hence ensure to add the app.properties file, and confirm the tests folder and other config files are included.

onyijne commented 4 years ago

please see before you proceed.

onyijne commented 4 years ago

on a second thought, since ionic start --type=react uses typescript (file extensions are ts) by default, and in other to build on the default js-template rather than removing it, you can first setup parcel with react, then add ionic.

onyijne commented 4 years ago

@Aliyugambo @habibkayode123 @nerjib @prominded lets point out what still needs to be done for the template to be in good to go, or approve if you think it’s okay.

onyijne commented 4 years ago

@Aliyugambo @habibkayode123 @nerjib @prominded lets point out what still needs to be done for the template to be in good to go, or approve if you think it’s okay.

Aliyugambo commented 4 years ago

from my point of view i think the template is good to go

prominded commented 4 years ago

Hopefully we ll close all pending frontend issues by tomorrow sunday 16th may.

I think a few dependencies we need to specify in d package.json file to get us fully ready.

I m trying to conclude d best approach to state management in React given d perculiarities of d project, after which we can tie all d pieces together.

prominded commented 4 years ago

@onyijne, are there still any specific concerns regarding the answers provided by @Aliyugambo?

Please lets know so we can attend to it

onyijne commented 4 years ago

@onyijne, are there still any specific concerns regarding the answers provided by @Aliyugambo?

Please lets know so we can attend to it

Regarding what, I don’t understand what you’re talking about.

Aliyugambo commented 4 years ago

@ttl why is this issue still not close. since all have been done i mean the necessary research are all done.

onyijne commented 4 years ago

@Ttl why is this issue still not close. since all have been done i mean the necessary research are all done.

PR#7 is still remaining one review to be merged. same as Phenom-Backend-PR#6