Skip to content
This repository has been archived by the owner on Jun 16, 2021. It is now read-only.

CSOC Task 4 Complete #13

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

CSOC Task 4 Complete #13

wants to merge 16 commits into from

Conversation

king-11
Copy link
Member

@king-11 king-11 commented Jun 2, 2020

All task completed 3rd party plugins used are :

  • sqflite
  • path_provider
  • intl
  • image_picker
  • shared_preferences

@king-11
Copy link
Member Author

king-11 commented Jun 5, 2020

@nishantkr18 I can't find a way to get past checks can you please tell what should i do.

@nishantkr18
Copy link
Member

Hey @king-11 impressive work! The extra features are really nice. Also, its good that u divided up some parts of your code.
The only minor issue I could find was the missing of an exit confirm. U can use WillPopScope for that easily.

For build issues, I'll send the instructions soon. Keep up the good work :)

@king-11
Copy link
Member Author

king-11 commented Jun 7, 2020

@nishantkr18 thanks for the review, I will use your suggestion in the upcoming app project that i have thought of for sure .

@nishantkr18
Copy link
Member

Thanks for participating! I've made changes to github workflows now, so go ahead and merge master into your branch if you want to have your test fixed. You should add all your work inside a directory named gmailapp, otherwise the tests will fail. Github actions will build your apk directly from your source files, and upload it as artifacts. You can visit your test after it passes, to downoad your built apk. For any queries, drop a message on the telegram group or pm.

And lastly, we have been working on the institute app for some time, and uses flutter. Here's the link. Currently, the work is on hold, but we shall continue after examinations. If you are interested to work, feel free to build the apk locally, add an issue if you find a bug, fix existing issues or add documentation. For backend, we use https://workshops-app-backend.herokuapp.com/ from here.

Keep contributing and stay safe.

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

Successfully merging this pull request may close these issues.

None yet

2 participants