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

May 6, 2020 #29

Merged
merged 6 commits into from
May 6, 2020
Merged

May 6, 2020 #29

merged 6 commits into from
May 6, 2020

Conversation

alpita-masurkar
Copy link
Owner

Description:

Linked issues:

Screenshots:

How to test:

martingd and others added 6 commits May 4, 2020 10:42
New option to dev_setup.sh, --skip-ios, to skip configuration related to iOS.
If Xcode is not installed, configuring without this option is not allowed.

 ### Testing Notes ###
Please run ./dev-setup.sh on a Mac without Xcode. Result should be:
  Xcode must be installed to develop for iOS.
  Use option '--skip-ios' to setup development for Android only.
  (exit code 1)
Add option --skip-ios and script should complete setting up
development for Android only.

Please run ./dev-setup on Mac _with_ Xcode and the script should run
like before and setup for both Android and iOS development.

 ### Fixed Issues ###
Setup script dev_setup.sh failed on macOS/Darwin if tools for iOS
development were not installed.
* pass 1 - change tripleblindmarket to Path-Check

* change to Path-Check/safeplaces-frontend where appropriate

* stronger wording on the pull request template

* license update

* more wordsmith

* undo license file change

Co-authored-by: Ken Pugsley <>
* Consolidate on single theme aware button

* Add secondary button style

* Fix crash

* Use the button everywhere!

* Button disabled states

* Fix onboarding1

* Fix bad merge

* Convert share button to <Button>

* Pass through style and other props

* More style updates and reconfig

* Button tweaks

* More tweaks

* Don't force opacity when not needed

* Add tests

* Update other snaps

* Fix iOS

* Add some spacing for iOS

* Restore padding for icon buttons

* update snaps
@alpita-masurkar alpita-masurkar merged commit 714286e into alpita-masurkar:develop May 6, 2020
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.

4 participants