wvsu-cict-code / thesis-package

College's guidelines in packaging your thesis application.
2 stars 4 forks source link

Macorol et. al. Package Evaluation #26

Closed cictapps closed 1 year ago

cictapps commented 1 year ago

Title: Lagaw Ta

Members Macorol, Pierre Angelo, Ocon, Max Ellison , Planco,Jam Roanne, Uy,John Rey, Young, Ma. Chelsea Joy

cictapps commented 1 year ago

Good day! Here are the feedback after checking your Github repository: Mobile app: No problems installing, tested the application on a Samsung A30s User manual:

  1. Spelling errors: please fix the spellings error present in your user manual, here are the few that I found: image Change to React Native image Change to Navigation image Change to Updating Tip: you can use the Grammarly plugin on Word (if you used it) to check and spot for spelling and grammatical errors.

  2. Installation requirements: Remove Python from the requirements since it is not used for the mobile app development. As for React Native, it needs additional software (Android Studio and SDK, NPM. You can follow the minimum requirements here for running React Native apps: apps here: https://learn.microsoft.com/en-us/windows/dev-environment/javascript/react-native-for-android

Also include the minimum requirements to run your mobile application

  1. Running the mobile app image Revise the commands for running the mobile app. you don't have a production/development server. For installing dependencies, the command is: yarn install Running the mobile app: yarn android OR react-native run-android Createing apk/appbundle: yarn build:android / yarn bundle:android

  2. If you have a keystore file generated, please put that inside the keys folder so that we can create and deploy the app.

Uyjohn commented 1 year ago

Keystore file uploaded already. To follow user manual edits. Thank you!

Uyjohn commented 1 year ago

Updated the User Manual Guide with revisions. Please check. Thank you!

cictapps commented 1 year ago

Good day @Uyjohn, all of the changes are present on the Github repo. @mjsolidarios changing the status for certification so that we can process the certificate in advance. Please upload the soft copy of your manuscript once it has passed the final checking, thank you!

cictapps commented 1 year ago

We have sent the completion certificate. Congratulations!