-
Notifications
You must be signed in to change notification settings - Fork 22
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
[PR] Adding deployment guide #343
base: main
Are you sure you want to change the base?
Conversation
I'm still in the process of detailing the deployment info. I've added a guide on The strangest thing is happening when I try to enrol into the I've tried looking for answers (such as https://developer.apple.com/forums/thread/723431) but nothing has worked. Perhaps Pi-hole is messing with redirection? |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #343 +/- ##
=======================================
Coverage 93.51% 93.51%
=======================================
Files 24 24
Lines 447 447
=======================================
Hits 418 418
Misses 29 29 ☔ View full report in Codecov by Sentry. |
@LuchoTurtle connect directly to the MEO router if PiHole is causing you any issues. 💭 |
… and not errors out." This reverts commit 6846822. # Conflicts: # pubspec.yaml
I'm almost finishing up the Android part of the guide. All of these have been documented and I've tried testing a release roll-out and it worked (see https://github.com/dwyl/app/actions/runs/6618383576/job/17977095655). The secrets that are currently present in this repository are my own/related to my account on Google Play. It may be better for DWYL to register as an organization and invite me as a collaborator so these variables are changed accordingly. |
I've finalised the Apple section and written a workflow, as well. |
@@ -73,6 +73,7 @@ jobs: | |||
uses: r0adkll/upload-google-play@v1.1.2 | |||
with: | |||
# MUST match the package name defined in `android/app/build.gradle` | |||
# Change all instances of of the package name to this one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. 🙏 ❤️
closes #342