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

🚀[Release v3.6.0] Merge into Main #91

Merged
merged 129 commits into from
Sep 25, 2021
Merged

🚀[Release v3.6.0] Merge into Main #91

merged 129 commits into from
Sep 25, 2021

Conversation

kcw-grunt
Copy link
Contributor

@kcw-grunt kcw-grunt commented Sep 24, 2021

Release Notes

Features

  • Added new logo
  • Add litecoin card transfer capability

Tech Debt

  • Added more tests

mosadialiou and others added 30 commits March 19, 2020 18:19
- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade
* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message
* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4f.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>
* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore
* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13
…on (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps
* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4f.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* [Release v2.8.2] Merge into Develop

## Release Notes: v2.8.2 (1)
This release is due to a requirement to App Store to remove references to donations.  In it’s replacement, a reference to the Litecoin Foundation support page where the user can elect to send LTC if they so choose.
- [Warmfix] Issue:146 Remove direct donation: support litecoin foundati…

##  Other Improvements
- Fix crash in Amount formatter (#140)
- [Bugfix] Renamed enum from State (#147) (#150)

## Bugfixes
 - Renaming the enum caused a conflict with SwiftUI @State

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Hotfix Increment

-Updated to allow upload to App Store

* build number change

* fixed conflicts

Co-authored-by: Mohamed Barry <[email protected]>
* Moved the LF Cell from Send.

- BartyCrouch linted the stigns files.

* Updated the LF Support location

- Reset the localiaztion language to ‘Customer support’
- Add the dismissal actions
* Updated the url to new servers

* Linted the Localized strings

* Setup url constants

* updated pk file status

* Updated per PR review comments

* fixed naming of the url variable
* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4f.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* [Release v2.8.2] Merge into Master  (#153)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* Develop - Release/v2.8.0 (#132)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4f.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* Fix crash in Amount formatter (#140)

* 🦺 [Tech Debt] Update Podfile and Refactor build process (#144)

* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore

* [Bugfix] Renamed enum from State (#147)

* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13

* Revert "[Bugfix] Renamed enum from State (#147)" (#150)

This reverts commit ea8293b.

* [Warmfix] Issue:146 Remove direct donation: support litecoin foundation (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps

* [Release v2.8.2] Merge into Develop

## Release Notes: v2.8.2 (1)
This release is due to a requirement to App Store to remove references to donations.  In it’s replacement, a reference to the Litecoin Foundation support page where the user can elect to send LTC if they so choose.
- [Warmfix] Issue:146 Remove direct donation: support litecoin foundati…

##  Other Improvements
- Fix crash in Amount formatter (#140)
- [Bugfix] Renamed enum from State (#147) (#150)

## Bugfixes
 - Renaming the enum caused a conflict with SwiftUI @State

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Hotfix Increment

-Updated to allow upload to App Store

* build number change

* fixed conflicts

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>

* version bump

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>
* Added UD to Podfile

- Stubbed out the view, viewModel and the test class for UD

* [UI Polish] Added images and localized strings

- Added more of strings and UD structure
- Trimmed images to resize for button
- Added localized strings for placeholder

* Updated UnstoppableDomainsView and Model details

- Added DEV notes script
- Adjusted Previews by unchecking the `build install only`

* Added more tests

- UnstoppableDomains View Model tests
- SupportLitecoinFoundationViewModel tests

* reformatted code in new SwiftUI views and viewModels

* Added UD action to the SendViewController

* Updated project

* Updated localized UD Placeholder

* Setup API keys

* removed pk file

* Changed the localizations related to UD

- Added ‘Or’ label
- Added UD placeholder text
- Changed the LTC address placeholder text

* Added in the Or and relayout of the button

- Added Partner layout view for UD and Simplex

* Resolved code review comments

- Update the localizations
- Refactored the UD shared instance (singleton) to retain throguh the call
- Added timing probes

* revert to current  version

* reformatted the Support LF view strings

* Remove Test Playground.

* Removed unused SocialView and ViewModel

* Improved resolution

- Added a Dispatch and ‘exit’ when address is found sooner than 12 secs. Much better experience for the user.

* Added infura

* Resolved review comments

* Rewrote copy for “Enter a Litecoin address”
# Conflicts:
#	Podfile
#	loafwallet.xcodeproj/project.pbxproj
#	loafwallet/SupportLitecoinFoundationView.swift
#	loafwallet/SupportLitecoinFoundationViewModel.swift
#	loafwallet/SupportSafariView.swift
#	loafwallet/SupportSafariViewModel.swift
#	loafwallet/src/Constants/Constants.swift
#	loafwallet/src/ModalPresenter.swift
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift
#	loafwalletTests/Constants Tests/ConstantsTests.swift
* Fix some triival conflict markers

* Added test to button once the UD string meets a minimum length

* Clears address text field when Lookup button is pressed

* Set the background color of the AlertToast

* Locailization of the alert language

* move logic to release the first responder

* per review notes
* [Litecoin Card] Progress Step 1

- Major work: litecoin-card TabBarView refactor
- Updated UIColor for Color
- SwiftUI preview devices
- Working out the  Nav polish
- Fixed hard-coded UD lookup label with proper localizations
- Commented out BitId

* [Litecoin Card] Progress Step 2

- Added more  localizations
- Rebuilding registration view need to add keyboard
- Fixed the animation for the card view
- Fixed Alert name collisions to the SimplexAlert

* [Litecoin Card] Progress Step 3

- Added Login views and localizations
- Added partnerAPI Swift Package
- Layout the registration view
- Add more localizations
- BUGFIX: Image missing in this branch.

* [Litecoin Card] Progress Step 4

More localizations
Added data Validators
Final Polish for Registration View
Registration View Modal wiring works

* [Litecoin Card] Progress Step 5

Register works to the model
Login is working
Added animation for Login/Logout
Login and Loogut working
Added localizations
More localizations

* [Litecoin Card] RC 1 polishes

Communcations polishing + dev cleanup
Polishing the RegistrationView layout
Fixed bug: #175

* [Litecoin Card] RC2: removed defunct script

* per review comments

- Ar files removal
- Cleanup localizable
- Removed Ar.proj files
- Removed empty class
- Added all Validation localization framework
- Added localizations of the validations
- Refactored per review notes

* Bugfix- login message

Added failure message localizations
kcw-grunt and others added 25 commits May 6, 2021 18:23
…keys-files-43

🦺[Tech debt] Add develop & GS keys file 43
* - Added tests for the Card test group

- Added test at launch of locale

* Added NonUSTabBar

Implemented switching of the tabs

* Checked that basic switching works

- Added a EXUS ViewContoller
- Added the setup code for both EXUS and TabBar ViewContollers
- Basic tests

* Added cell for locale

Added the Locale Detail view

* Added “Current Locale” localizations

* Polished tests, added locale detail view

- Clean up of code
- Added SwiftUI Locale Detail View

* pre review comments

- meeting switch block indentation standards

* per review comments.

* removed flaky test
* Added templates

fix template

* fixed spike
* Removed price label from Header View

- Extended tap area of the balance animation

* removed dev loginviewcontroller storyboard

* Added new lock screen SwiftUI view

* Added localizations for the Lock Screen (Login)

* Added SwiftUI : LockScreen HeaderView

- Dev code fixes
- Adjust layout

* Added simple test for the Lock Screen symbol
* [Merge v3.0.0] into Master (#183)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* Develop - Release/v2.8.0 (#132)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* Fix crash in Amount formatter (#140)

* 🦺 [Tech Debt] Update Podfile and Refactor build process (#144)

* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore

* [Bugfix] Renamed enum from State (#147)

* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13

* Revert "[Bugfix] Renamed enum from State (#147)" (#150)

This reverts commit ea8293b5c7f206706942416a1d7d9b139926fa97.

* [Warmfix] Issue:146 Remove direct donation: support litecoin foundation (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps

* [Release] Merge v2.8.2 into Develop (#158)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* [Release v2.8.2] Merge into Develop

## Release Notes: v2.8.2 (1)
This release is due to a requirement to App Store to remove references to donations.  In it’s replacement, a reference to the Litecoin Foundation support page where the user can elect to send LTC if they so choose.
- [Warmfix] Issue:146 Remove direct donation: support litecoin foundati…

##  Other Improvements
- Fix crash in Amount formatter (#140)
- [Bugfix] Renamed enum from State (#147) (#150)

## Bugfixes
 - Renaming the enum caused a conflict with SwiftUI @State

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Hotfix Increment

-Updated to allow upload to App Store

* build number change

* fixed conflicts

Co-authored-by: Mohamed Barry <[email protected]>

* [Bugfix] Replace the LF Support View (#159)

* Moved the LF Cell from Send.

- BartyCrouch linted the stigns files.

* Updated the LF Support location

- Reset the localiaztion language to ‘Customer support’
- Add the dismissal actions

* [Bugfix] Fix buy tab: Simplex urls (#157)

* Updated the url to new servers

* Linted the Localized strings

* Setup url constants

* updated pk file status

* Updated per PR review comments

* fixed naming of the url variable

* [Merge v2.8.3] into Develop (#169)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* [Release v2.8.2] Merge into Master  (#153)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* Develop - Release/v2.8.0 (#132)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* Fix crash in Amount formatter (#140)

* 🦺 [Tech Debt] Update Podfile and Refactor build process (#144)

* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore

* [Bugfix] Renamed enum from State (#147)

* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13

* Revert "[Bugfix] Renamed enum from State (#147)" (#150)

This reverts commit ea8293b5c7f206706942416a1d7d9b139926fa97.

* [Warmfix] Issue:146 Remove direct donation: support litecoin foundation (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps

* [Release v2.8.2] Merge into Develop

## Release Notes: v2.8.2 (1)
This release is due to a requirement to App Store to remove references to donations.  In it’s replacement, a reference to the Litecoin Foundation support page where the user can elect to send LTC if they so choose.
- [Warmfix] Issue:146 Remove direct donation: support litecoin foundati…

##  Other Improvements
- Fix crash in Amount formatter (#140)
- [Bugfix] Renamed enum from State (#147) (#150)

## Bugfixes
 - Renaming the enum caused a conflict with SwiftUI @State

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Hotfix Increment

-Updated to allow upload to App Store

* build number change

* fixed conflicts

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>

* version bump

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>

* [Feature Issue 154] Adding Unstoppable Domains functionality  (#155)

* Added UD to Podfile

- Stubbed out the view, viewModel and the test class for UD

* [UI Polish] Added images and localized strings

- Added more of strings and UD structure
- Trimmed images to resize for button
- Added localized strings for placeholder

* Updated UnstoppableDomainsView and Model details

- Added DEV notes script
- Adjusted Previews by unchecking the `build install only`

* Added more tests

- UnstoppableDomains View Model tests
- SupportLitecoinFoundationViewModel tests

* reformatted code in new SwiftUI views and viewModels

* Added UD action to the SendViewController

* Updated project

* Updated localized UD Placeholder

* Setup API keys

* removed pk file

* Changed the localizations related to UD

- Added ‘Or’ label
- Added UD placeholder text
- Changed the LTC address placeholder text

* Added in the Or and relayout of the button

- Added Partner layout view for UD and Simplex

* Resolved code review comments

- Update the localizations
- Refactored the UD shared instance (singleton) to retain throguh the call
- Added timing probes

* revert to current  version

* reformatted the Support LF view strings

* Remove Test Playground.

* Removed unused SocialView and ViewModel

* Improved resolution

- Added a Dispatch and ‘exit’ when address is found sooner than 12 secs. Much better experience for the user.

* Added infura

* Resolved review comments

* Rewrote copy for “Enter a Litecoin address”

* Fix some trivial conflict markers (#177)

* [Bugfix] Upgrade the business logic for Unstoppable Domains (#179)

* Fix some triival conflict markers

* Added test to button once the UD string meets a minimum length

* Clears address text field when Lookup button is pressed

* Set the background color of the AlertToast

* Locailization of the alert language

* move logic to release the first responder

* per review notes

* [Feature] Litecoin card v1 (#180)

* [Litecoin Card] Progress Step 1

- Major work: litecoin-card TabBarView refactor
- Updated UIColor for Color
- SwiftUI preview devices
- Working out the  Nav polish
- Fixed hard-coded UD lookup label with proper localizations
- Commented out BitId

* [Litecoin Card] Progress Step 2

- Added more  localizations
- Rebuilding registration view need to add keyboard
- Fixed the animation for the card view
- Fixed Alert name collisions to the SimplexAlert

* [Litecoin Card] Progress Step 3

- Added Login views and localizations
- Added partnerAPI Swift Package
- Layout the registration view
- Add more localizations
- BUGFIX: Image missing in this branch.

* [Litecoin Card] Progress Step 4

More localizations
Added data Validators
Final Polish for Registration View
Registration View Modal wiring works

* [Litecoin Card] Progress Step 5

Register works to the model
Login is working
Added animation for Login/Logout
Login and Loogut working
Added localizations
More localizations

* [Litecoin Card] RC 1 polishes

Communcations polishing + dev cleanup
Polishing the RegistrationView layout
Fixed bug: https://github.com/litecoin-foundation/loafwallet-ios/issues/175

* [Litecoin Card] RC2: removed defunct script

* per review comments

- Ar files removal
- Cleanup localizable
- Removed Ar.proj files
- Removed empty class
- Added all Validation localization framework
- Added localizations of the validations
- Refactored per review notes

* Bugfix- login message

Added failure message localizations

* version bump

* Added some tests

* Refactored the login logic

- Used the status of the binding to change the UI

* Added combined logic  of the email and password fields to enable login button

Reference @losh11 comment:
`The cards login now gives a good login failed alert. However I think the login button should be disabled until the text in the email input is validated as email format.`

* build bump

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>

* [HOTFIX] Hide Card Tab (#191)

* Hide Card Tab

* added comments for v1

* [Merge into Master] v3.1.1  (#194)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* Develop - Release/v2.8.0 (#132)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* Fix crash in Amount formatter (#140)

* 🦺 [Tech Debt] Update Podfile and Refactor build process (#144)

* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore

* [Bugfix] Renamed enum from State (#147)

* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13

* Revert "[Bugfix] Renamed enum from State (#147)" (#150)

This reverts commit ea8293b5c7f206706942416a1d7d9b139926fa97.

* [Warmfix] Issue:146 Remove direct donation: support litecoin foundation (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps

* [Release] Merge v2.8.2 into Develop (#158)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* [Release v2.8.2] Merge into Develop

## Release Notes: v2.8.2 (1)
This release is due to a requirement to App Store to remove references to donations.  In it’s replacement, a reference to the Litecoin Foundation support page where the user can elect to send LTC if they so choose.
- [Warmfix] Issue:146 Remove direct donation: support litecoin foundati…

##  Other Improvements
- Fix crash in Amount formatter (#140)
- [Bugfix] Renamed enum from State (#147) (#150)

## Bugfixes
 - Renaming the enum caused a conflict with SwiftUI @State

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Hotfix Increment

-Updated to allow upload to App Store

* build number change

* fixed conflicts

Co-authored-by: Mohamed Barry <[email protected]>

* [Bugfix] Replace the LF Support View (#159)

* Moved the LF Cell from Send.

- BartyCrouch linted the stigns files.

* Updated the LF Support location

- Reset the localiaztion language to ‘Customer support’
- Add the dismissal actions

* [Bugfix] Fix buy tab: Simplex urls (#157)

* Updated the url to new servers

* Linted the Localized strings

* Setup url constants

* updated pk file status

* Updated per PR review comments

* fixed naming of the url variable

* [Merge v2.8.3] into Develop (#169)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* [Release v2.8.2] Merge into Master  (#153)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* Develop - Release/v2.8.0 (#132)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* Fix crash in Amount formatter (#140)

* 🦺 [Tech Debt] Update Podfile and Refactor build process (#144)

* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore

* [Bugfix] Renamed enum from State (#147)

* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13

* Revert "[Bugfix] Renamed enum from State (#147)" (#150)

This reverts commit ea8293b5c7f206706942416a1d7d9b139926fa97.

* [Warmfix] Issue:146 Remove direct donation: support litecoin foundation (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps

* [Release v2.8.2] Merge into Develop

## Release Notes: v2.8.2 (1)
This release is due to a requirement to App Store to remove references to donations.  In it’s replacement, a reference to the Litecoin Foundation support page where the user can elect to send LTC if they so choose.
- [Warmfix] Issue:146 Remove direct donation: support litecoin foundati…

##  Other Improvements
- Fix crash in Amount formatter (#140)
- [Bugfix] Renamed enum from State (#147) (#150)

## Bugfixes
 - Renaming the enum caused a conflict with SwiftUI @State

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Hotfix Increment

-Updated to allow upload to App Store

* build number change

* fixed conflicts

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>

* version bump

Co-authored-by: Mohamed Barry <[email protected]>
Co-authored-by: Nikolay Spassov <[email protected]>

* [Feature Issue 154] Adding Unstoppable Domains functionality  (#155)

* Added UD to Podfile

- Stubbed out the view, viewModel and the test class for UD

* [UI Polish] Added images and localized strings

- Added more of strings and UD structure
- Trimmed images to resize for button
- Added localized strings for placeholder

* Updated UnstoppableDomainsView and Model details

- Added DEV notes script
- Adjusted Previews by unchecking the `build install only`

* Added more tests

- UnstoppableDomains View Model tests
- SupportLitecoinFoundationViewModel tests

* reformatted code in new SwiftUI views and viewModels

* Added UD action to the SendViewController

* Updated project

* Updated localized UD Placeholder

* Setup API keys

* removed pk file

* Changed the localizations related to UD

- Added ‘Or’ label
- Added UD placeholder text
- Changed the LTC address placeholder text

* Added in the Or and relayout of the button

- Added Partner layout view for UD and Simplex

* Resolved code review comments

- Update the localizations
- Refactored the UD shared instance (singleton) to retain throguh the call
- Added timing probes

* revert to current  version

* reformatted the Support LF view strings

* Remove Test Playground.

* Removed unused SocialView and ViewModel

* Improved resolution

- Added a Dispatch and ‘exit’ when address is found sooner than 12 secs. Much better experience for the user.

* Added infura

* Resolved review comments

* Rewrote copy for “Enter a Litecoin address”

* Fix some trivial conflict markers (#177)

* [Bugfix] Upgrade the business logic for Unstoppable Domains (#179)

* Fix some triival conflict markers

* Added test to button once the UD string meets a minimum length

* Clears address text field when Lookup button is pressed

* Set the background color of the AlertToast

* Locailization of the alert language

* move logic to release the first responder

* per review notes

* [Feature] Litecoin card v1 (#180)

* [Litecoin Card] Progress Step 1

- Major work: litecoin-card TabBarView refactor
- Updated UIColor for Color
- SwiftUI preview devices
- Working out the  Nav polish
- Fixed hard-coded UD lookup label with proper localizations
- Commented out BitId

* [Litecoin Card] Progress Step 2

- Added more  localizations
- Rebuilding registration view need to add keyboard
- Fixed the animation for the card view
- Fixed Alert name collisions to the SimplexAlert

* [Litecoin Card] Progress Step 3

- Added Login views and localizations
- Added partnerAPI Swift Package
- Layout the registration view
- Add more localizations
- BUGFIX: Image missing in this branch.

* [Litecoin Card] Progress Step 4

More localizations
Added data Validators
Final Polish for Registration View
Registration View Modal wiring works

* [Litecoin Card] Progress Step 5

Register works to the model
Login is working
Added animation for Login/Logout
Login and Loogut working
Added localizations
More localizations

* [Litecoin Card] RC 1 polishes

Communcations polishing + dev cleanup
Polishing the RegistrationView layout
Fixed bug: https://github.com/litecoin-foundation/loafwallet-ios/issues/175

* [Litecoin Card] RC2: removed defunct script

* per review comments

- Ar files removal
- Cleanup localizable
- Removed Ar.proj files
- Removed empty class
- Added all Validation localization framework
- Added localizations of the validations
- Refactored per review notes

* Bugfix- login message

Added failure message localizations

* [Merge v3.0.0] into Develop (#184)

* version bump

* Added some tests

* Refactored the login logic

- Used the status of the binding to change the UI

* Added combined logic  of the email and password fields to enable login button

Reference @losh11 comment:
`The cards login now gives a good login failed alert. However I think the login button should be disabled until the text in the email input is validated as email format.`

* build bump

* [HOTFIX]  hide litecoin card (#192)

* [Merge v3.0.0] into Master (#183)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* Develop - Release/v2.8.0 (#132)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <[email protected]>

* Fix crash in Amount formatter (#140)

* 🦺 [Tech Debt] Update Podfile and Refactor build process (#144)

* Update Podfile

- Set mimum value of iOS 13
- Removed unused SwiftyJSON
- Removed unused Watch Scheme
- re-added xcscheme
- removed non-used code
- added status badge in README

* removed the podfile target force

* added SPM .build folder exclusion in gitignore

* [Bugfix] Renamed enum from State (#147)

* Renamed enum from State

 ## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState`

* Toggled iOS version 13

* Revert "[Bugfix] Renamed enum from State (#147)" (#150)

This reverts commit ea8293b5c7f206706942416a1d7d9b139926fa97.

* [Warmfix] Issue:146 Remove direct donation: support litecoin foundation (#151)

* Prepared the code for differnt Cell / Remove Donation Code

- Updated localized strings file(s)
- Removed old Donate files
- Updated SendVIewController

* Resolved the previous conflicts

- Added in SupportLitecoinFoundation SwiftUI view and model
- Working in the basic functionality of the new Support Litecoin Foundation view

* Worked in the vars of SupportLitecoinFoundation  view

* Removed Donation code

* Set a URL for the support LTC address

* Add new WebKit view and supporting code

* Adjusted height constants in the nightmare scenario of the SendCell

-There is a nasty sandwich of ViewControllers where SwiftUI should be used.
- This needs to be simplified

# Conflicts:
#	loafwallet/src/ViewControllers/RootModals/SendViewController.swift

* Added new MVVM Views and ViewModels

- Adding more SwiftUI code

* Refactored and stubbed out tests

* version bump

* Renamed enum from State  to ReduxState

## Problem
 The original design of Breadwallet (Loafwallet) used a SimpleRedux architecture which maanged the State in an unsual (yet effective) way.

Within that design ‘state’ was managed by an enum named “State”

This was fine until SwiftUI and Combine where there is now a name collision since @State is referring to the SwfitUI attribute.

 ## Approach
While redesiging the app would be ideal, the reality is this needs to be fixed as soon as possible because Litewallet is now using SwiftUI.

 Steps to fix the problem:
 - [ ] Renamed variable from `State` to `ReduxState` using Xcode Refactor: Rename

* Cleaned up Localized strings file

* Added review fixes

- Added CGFloat multiplier in padding
- Fixed formatting
- Added tracking Support taps

* [Release] Merge v2.8.2 into Develop (#158)

* [Merge to Master] Develop with tech debt improvements (#123)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

Co-authored-by: Mohamed Barry <[email protected]>

* Release/v2.8.0 (#131)

* fix crash: window required

* compute QR code only once

* run code on main thread

* update cocoapods firebase

* update addresses + event

* version bump

* remove unused fonts

* Updated key for RemoteRunnable in xcscheme

* Added Issue templates (#122)

- Bug Report
- Feature Request
- Also have watch xcscheme added per upgrade

* [Bugfix] Cleared Simplex  (#126)

* Disabled Simplex 
- Was causing crashes when return messages were happening.
- Will update in the next cycle with an updated UI
- Added a Analytics:  DID TAP BUY TAB marker

* Added Coming Soon label to the tableview background view for a temporary sign
- Added all translation for the coming soon message

* ## Release Notes: v2.8.0  (version bump)

This release is the first release in months that needed an update based on the changes in the enviroment. In general, the Litewallet code base is bloated with 1000's of lines of unused code and this makes debugging and adding new features very difficult. In addition, the requirements of iOS 14 and the plan to move from UIKit to a SwiftUI+Combine architecture means subsequent PRs will reflect a more streamlined codebase.

## Tech Debt
- Also have watch xcscheme added per upgrade
- Added Issue templates (#122)
	- Bug Report
	- Feature Request
- Updated key for RemoteRunnable in xcscheme
- remove unused fonts
- run code on main thread
- update cocoapods firebase
- update addresses + event

## Improvements
* compute QR code only once
	- Soft launch for Import QR Code

## Bugfixes
* fix crash: window required

## Temporary Workarounds
The proxy server the supports buying LTC is being refactored and so no mobile clients can use the Simplex service
	* Disabled Simplex / Added temp message
		- Was causing crashes when return messages were happening.
		- Will update in the next cycle with an updated UI
		- Added a Analytics:  DID TAP BUY TAB marker
		- Added Coming Soon label to the tableview background view for a temporary sign
		- Added all translation for the coming soon message

## Authors
Kerry Washington <[email protected]>
Mohamed Barry <[email protected]>

* Removed schemes (#125)

Goal: Reduce the codebase in prep for a major refactor. This removed the debug scheme and watch scheme.

* [Tech Debt] Remove Apple Watch code (#124)

* Remove  Apple Watch code

- The Apple Code is very old and has never been updated.
- There is a longer plan to remove superflous code.
- If others ask to re-add it we will look again with the later versions

* Removed NotificationServiceExtension

- This is cruft that has never been called
- Added in 2016
- This is not supported currently

* Change email addresses

Also added feedback and support emails.

* Added HTML for support email

Fixed bitcoin references

* Removed empty UITests

* bump build number

* updated html to be for iPhone only

* Fix podfile and build version

* Revert "[Bugfix] Cleared Simplex  (#126)"

This reverts commit 154eb4fbbe71b4ff1a2323dd316bf7b6783c3b0a.

* bump build number

Co-authored-by: Mohamed Barry <mos…
* Hotfix: Set item tags to the proper sequence

* Refactored tests

- Added TabBarViewController Tests
- Added NonUSTabBarViewController Tests
- Fixed Default Currency Test

* version bump
* Added release template

* per review comments

- Fixed typos, mispellings and slight change of meanings
* Changed to match iosdev

* simple CHANGENOW for test

* Changes are unfortunately not trivial apparently.

Workspace file changes and automatic changes by the xcode environment.

* reverted project file changes

* tiny change in the CHANGELOG for testing

* Trivial change to check kcw-grunt and myself are in sync

Co-authored-by: Jasper Heist <[email protected]>
Turkish language support added by translating strings using Google Translate.
- Turkish is now found
* Removed the launch options

- This  method had  `launchOptions`  that was never used.

* Removed all unused code.
* updated to the latest API library

- Moved to v0.8.0

* Remove (swiftch off) code coverage

- There is a bug in Xcode where running code coverage breaks the SwiftUI canvas

* Added the basic localization

- Pre-polish the languiage and the translations
- Tech Debt: Code clean up
- Deleted and cleaned up files.

* Updated the CardViewController

- UIKit updated relative to the modal

* First version of 2FA working

- New 2FA View
- New 2FA Model

* Cleaned up code formatting

* Polished the events and notes

* Added 2FA text

Localizations:

* 2FA Not Enabled
* 2FA Enabled
* Enter the code that was recently sent to your Litecoin Card account email.
* Enter Code
* Confirm

* Added tests

- Stubbed in the View Extenstion tests
- Enable2FAViewModel tests

* Resolved per PR notes

* Restruct localizations (#83)

- Turkish is now found

* 🦺[Tech debt] Refactor send class (#80)

* Removed the launch options

- This  method had  `launchOptions`  that was never used.

* Removed all unused code.
* Added the basic localization

- Pre-polish the languiage and the translations
- Tech Debt: Code clean up
- Deleted and cleaned up files.

* Updated the CardViewController

- UIKit updated relative to the modal

* First version of 2FA working

- New 2FA View
- New 2FA Model

* Cleaned up code formatting

* Polished the events and notes

* Added 2FA text

Localizations:

* 2FA Not Enabled
* 2FA Enabled
* Enter the code that was recently sent to your Litecoin Card account email.
* Enter Code
* Confirm

* Added tests

- Stubbed in the View Extenstion tests
- Enable2FAViewModel tests

* Resolved per PR notes

* Preliminary steps

- Added basic structure

* 🥳 Added PreTransfer View and Model

- Added mock data
- PreTransfer enum for Litewallet v Litecoin Card to make sure the images and the labels match
- Added basics for a viewModel
- Began adjusting the padding

* reverted Podfile to the `incorrect version`

* 🖍 Added Transfer UI

- Adding transfer button
- Added new label strings for Transfer
- Polsihed the pretransfer view
- Tech Debt: Added lighter gray to the color palette

* 🖍📐Major UI/UX updates for the Transfer functionalty

- Added standard padding
- Adjustment of views
- Padding adjustments
- Added the second stage to set the amount
- Changed the mock data for canvas and testing.
- Added more localizable strings

- Added Binding to trigger the animation: 🥳
Helpful articles:
* https://stackoverflow.com/questions/65831681/can-you-animate-a-swiftui-view-on-disappear
* https://www.vadimbulavin.com/passing-data-between-swiftui-views5cbbd211 - Adjusted strings content

* 🥳🖍📐Added UI/IX AmountView and the AmountViewModel

- Added border card image
- Changed to a Slider view to select amount
- Removed the disclaimer card toast
- Updated the strings / localizations
- Removed cruft and cleaned up test code.

* 🚧Refactored the CardViewController

- Udpated the subviews to the CardLoggedin View
- Added documentation and dev notes in the code

* ✨🧽 Added the Set Amoint UI for fine/coarse amount setting

- Added a simple Modal and toggle state to show
- Added disable/enable coloring of the Transfer button
- Added stepper buttons and a slider value change to match.
- Added the calculated the values to update the values
- FINALLY Done fixing the slider
- FINALLY Done fixing  the - and + buttons

* 🦟 Major debugging to meet the transfer business functionality

- Added changes about the UI/UX from Charlie discussion
- Litewallet pre-transfer always on top
- Add Card balance text in Transfer Amount View
- Add bold to the transfer to card amount
- Added new Localizations
- Added  card image
- Added Card Icon view
- Card Icon view structure
- Adjusted shadow of the icons

* 🦟Debugged displaying Transferering View Modal

- Made TransferringView work
- Converted to OK button
- Added the Publisher (Bool)
- Removed WalletBalanceStatus enum as it wasnt needed

* 🚧 🦺Tech Debt: Refacored code and made a general dev cleanup

- Added ObservableWallets : Card Balance property
- Added new localizations
- Added more event constants
- Reformatting
- Remove empty lines, etc.

* 🦺 Various developement fixes

- 🚧🚧Need to add the Card to Litewallet functionality
- Added localizations
- For No Transfer details
- Added Fragments
- Added 2FA messages
- Re-added the corner
- Added documetation for View Extension
- Added all localizations for the text
- Added code for when the user without 2FA

* 🦺Refactor: Dev code cleanup

- Removed cruft
- Removed unused code

* 🚧Refactor: 2FA

- Added new views for no  2FA
- Added a single observed object showing the 2FA status for all subscribers

* Added Testnet address

* ❄️Code Mothballed

- The process to withdraw from the Litecoin Card to Litewallet requires the User has logged in and must have 2FA enabled.
- Since that 2FA is enabled withdrawalToWallet should work...but the error from the Ternio API says must be 2FA Enabled.
- Once Ternio sorts the proper process and shares to the Litewallet team, we will add it.
- Meanwhile I have commented out the code.

* Per review notes
- Cleaned up podfile
- Turning off the code coverage allows the SwiftUI canvas to work
- Replaced all app icon assets
# Conflicts:
#	.gitignore
#	Podfile
#	loafwallet.xcodeproj/project.pbxproj
#	loafwallet.xcodeproj/xcshareddata/xcschemes/loafwallet.xcscheme
#	loafwallet.xcworkspace/xcshareddata/swiftpm/Package.resolved
#	loafwallet/AnimatedCardViewModel.swift
#	loafwallet/AppDelegate.swift
#	loafwallet/CardLoggedInView.swift
#	loafwallet/CardV1ToastView.swift
#	loafwallet/CardView.swift
#	loafwallet/CardViewController.swift
#	loafwallet/CardViewModel.swift
#	loafwallet/CardWalletDetails.swift
#	loafwallet/Color+Extension.swift
#	loafwallet/Debug-GoogleService-Info.plist
#	loafwallet/ForgotView.swift
#	loafwallet/LoginCardAlertView.swift
#	loafwallet/LoginViewModel.swift
#	loafwallet/RegistrationViewModel.swift
#	loafwallet/TabBarViewController.swift
#	loafwallet/View+Extension.swift
#	loafwallet/src/Constants/Constants.swift
#	loafwallet/src/Constants/Strings.swift
#	loafwallet/src/FlowControllers/URLController.swift
#	loafwallet/src/ModalPresenter.swift
#	loafwallet/src/Sender.swift
#	loafwallet/src/Strings/Base.lproj/Localizable.strings
#	loafwallet/src/Strings/da.lproj/Localizable.strings
#	loafwallet/src/Strings/de.lproj/Localizable.strings
#	loafwallet/src/Strings/en.lproj/Localizable.strings
#	loafwallet/src/Strings/es.lproj/Localizable.strings
#	loafwallet/src/Strings/fr.lproj/Localizable.strings
#	loafwallet/src/Strings/id.lproj/Localizable.strings
#	loafwallet/src/Strings/it.lproj/Localizable.strings
#	loafwallet/src/Strings/ja.lproj/Localizable.strings
#	loafwallet/src/Strings/ko.lproj/Localizable.strings
#	loafwallet/src/Strings/nl.lproj/Localizable.strings
#	loafwallet/src/Strings/pt.lproj/Localizable.strings
#	loafwallet/src/Strings/ru.lproj/Localizable.strings
#	loafwallet/src/Strings/sv.lproj/Localizable.strings
#	loafwallet/src/Strings/zh-Hans.lproj/Localizable.strings
#	loafwallet/src/Strings/zh-Hant.lproj/Localizable.strings
#	loafwallet/src/WalletManager.swift
#	loafwalletTests/Class Tests/LitecoinCardTests/CardWalletTests.swift
@kcw-grunt kcw-grunt marked this pull request as ready for review September 24, 2021 18:01
@kcw-grunt kcw-grunt requested a review from JCHeist September 24, 2021 18:01
Copy link
Contributor

@JCHeist JCHeist left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing jumps out. Looks good to me.

@kcw-grunt kcw-grunt changed the title 🚀[Release v3.6.0] Merge into Master 🚀[Release v3.6.0] Merge into Main Sep 25, 2021
@kcw-grunt kcw-grunt merged commit 864bc49 into main Sep 25, 2021
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.

3 participants