Skip to content
This repository was archived by the owner on Sep 20, 2023. It is now read-only.

Commit 1c40d45

Browse files
authored
Cut version 1.23.0 (#2380)
* Fix cell size on the Bookmarks tab (#2124) * Create code of conduct (#2126) It is encouraged by GitHub to comply with recommended community standards. * Fixed s/nuber/number/ typo in SECURITY.md (#2130) * Add issue template for feature request (#2132) Use GitHub template. * Add issue template for bug report (#2131) Use GitHub template. * bump version to 1.23 (#2143) * Replace "Inbox Zero" functionality and remove Firebase (#2142) * simpler inbox zero date that allows date planning * clean up * remove firebase * Don't override entire message when replying (#2160) * Local push notifications (#2145) * add fmdb * add local notification cache mechanism * rewiring to update local db when fetching notifications * local pushes working * building for xcode 10 * Add UIAppearance styling for UISwitch & UISearchBar (#2144) * Add(UIAppearance styling for UISwitch) * Add(UIAppearance for UISearchBar) * Added activity indicator (#2157) * Fixes bug where send button is enabled after sending a comment (#2158) * Load Consistency (#2159) * Combine 2 load cells into 1 * Fixed load more in BaseViewController * Switch if-else to ternary * adds review GitHubAccess button to notifications view controller (#2176) * Action Controller images (#2135) * Add images to action controller * Added pod to podfile Added icon to NotificationSectionController * Installed Pod * ContextMenu Dominant corner * Target support files * Fixed support * Updated Action Image Controller * grammatical change to code signing instructions (#2186) * Add TestFlight link to README (#2194) * Add TestFlight link to README * Update README.md * fix double tap tab invalid (#2192) this bug is caused by the time interval between two taps which is too small. * Fixed color issue. (#2188) * allows setting Milestone's loading indicator's color (#2195) * allows setting loading indicator's color * updated spinner color to .white in Milestones/Labels/People * Merge button status (#2189) * Addresses #2178 Shows merge status without button when you cannot merge. * Swift naming * Fortify local notifications code (#2191) * move path/defaults to be injected * remove unused defaults * add tests for notification db, fix settings, avoid dupe inbox requests * better tests, fix dupe insert bug * Attempt to fix iTunes Connect issue where watchOS needs min target (#2173) * attempt to fix iTunes Connect issue where watchOS needs min target * fix podfile * Use updatedAt time to key notification send times (#2201) * Use updatedAt time to key notification send times * better formatting and fix tests * Delete CNAME * move blog to own repo (#2205) * fixed icon name (#2203) * Move lock (#2206) * Moves lock under divider * fix param style * Update IssueManagingContextController.swift * Switched to constants (#2207) * move designs to own repo (#2208) * Delete appcenter-post-clone.sh * Delete AppStore.md * Restore readme images after assets move (#2210) * Move entitlements to Resources (#2209) * Fixed cell border on merge button (#2204) * Fixed cell border on merge button * Fixed nits * Settings cell textColor (#2193) * Set API Status cell textColor to custom (Matching the rest of the settings page) * Updated Double Tap Reaction off color to custom * Updated Enabled textColor to custom * Updated reaction textColors to proper custom * Switched all labels to settingsLabels * Fix entitlements path (#2211) * change repo branch (#2202) * change repo branch additions: - A change-repository-branch workflow thats very similar to the workflow in Milestones - GraphQL query to fetch a repo's branches modifications: - a mutable "branch" string in RepositoryOverviewController and RepositoryCodeDirectoryViewController - updated the V3RepositoryReadME fetch to include a branch parameter - a protocol RepositoryBranchUpdatable to flag and update the appropriate ViewControllers when a user switches branches - ContextMenu and UIAlertAction setup - Removed some outdated code to get a repo's branch name from fetch(page:) in RepositoryOverviewViewController - Added a line feed.adapter.reloadData() in fetch(page:) because app was crashing intermittently after a user switched branches without that line - was always an IGListKit duplicate identifier error on StyledTextRenderers - to reproduce, remove the feed.adapter.reloadData(), go to https://github.com/TheAlgorithms/Python, try to switch branches and the app will crash. I'm working raising an issue for it. * requested changes - changed public var to private(set) for var branch: String - removed wasteful feed.adapter.reloadData() call - re-formated params in switchBranchAction() * style nit * Localize Inbox Zero and allow for per-year holidays (#2215) * Update Setup.md (#2218) Grammatical change to md * set spinner color in RepoBranchVC (#2221) * Order repo branches after fetch (#2228) * Order repo branches after fetch - added func to order branches + unit tests - replaced 'var branch: String' w/ 'private(set) var selectedBranch: String' & added a defaultBranch variable - changed fetch(page:) to update(animated:) in didSelect(value:) - an oversight from original pr * AX animation * Fix readme duplicate identifier asserts (#2219) * WIP to fix readme asserts * remove newline * improved markdown parsing working * fix tests * Huge refactor of AppDelegate and authentication routing (#2238) * refactor with new app controller, replace root nav mgr * finish refactoring out root nav mgr * remove root mgr * Adds more share actions when browsing a repository (#2161) (#2237) * add additional share actions * fix share URLs * use edge inset to right align image (#2232) * new routing library and refactor shortcuts (#2241) * [FIX] Settings write review indent (#2242) * PeopleVC: Sort users with self first (#2246) - moves sorting logic from fetch(page:) into type method - unit test * Simpler logic for review access button handling (#2249) * Always update shortcuts on app launch (#2248) * move the bookmark icon for the RepoVC and address a CR comment (#2255) * always call completion block when early returning b/c of bg (#2258) * open issue from notification (#2259) * [FIX] Navigation for double tap reaction on settings (#2275) - Changes segue kind from show to showDetail - Adds a navigation controller for "Double tap Reaction" so it will work on both iPhone and iPad * [FIX] Navigation controller on "View Source" from settings (#2276) * keep read layer in front when animating inbox (#2280) * Proper fix to prevent double fetching from background (#2279) * Update Gemfile to use same version of CocoaPods referenced in Podfile.lock (#2277) * Update cocoapods version constraint * Run `bundle update cocoapods` * Run `npm install` * Add new iPhone models to UIDevice+Model.swift (#2278) * Switched from long press to tap recognizer for menu controllers (#2271) * Add steps in the setup to avoid commiting env variables to the repo (#2262) * Add "Try Again" button for EmptyView (#2214) (#2226) * Button to display Push Notification info in Settings (#2282) * Add push notification info in settings * better sizing * add empty error view to latest base VC (#2283) * move background handling into feed for better state control (#2285) * thread error descriptions to squawk throughout app (#2286) * Update MessageViewController (#2287) * update messageviewcontroller * add latest update * update with revert * Add clear button to action menus (#2288) * Add clear button to the labels and milestone action menu * Move the selection count in the PeopleViewController into the title Add a clear button to the people controller * Remove mock return for testing action menu * Add "Clear" string constant Remove unused Protocol * Setup the clear enabled state and the Poeple title correctly on init * Sorts issues after fetch (#2304) *reverse chronologically * Warn that logging out removes bookmarks (#2303) * Enable SwiftLint and fix issues (#2292) * update swiftlint * build with lint enabled * fix almost all warnings * remove wholemodule * fetch subscription status and fix mutation (#2291) * update ContextMenu (#2310) * reopen menu item is green (#2311) * fix landscape read animation issue (#2312) * Search for labels in app (take two) (#2314) * Created DidTap Protocol for label section controllers - created a protocol to route tap events to a delegate. The idea is that the delgate will have a reference to GithubClient and we can limit the baggage of passing around GithubClient * Refactor Repo VC and SC to use name and owner as parameters - Will make it easier to present RepoVC without needing a ref to RepositoryDetails. The only parameters being used from RepositoryDetails were owner and name * Added presentLabels extension to UIViewController * requested changes * Show PR CI status inline with title (#2325) * remove unused gql * show PR CI status in list * update StyledTextKit * Thread networking errors and add custom error descriptions (#2324) * Better network error descriptions * localized descriptions for custom errors * Improve empty retry UX (#2323) * Highlight CodeView text on a background queue (#2322) * Browse commit history of repositories, directories, and files (#2321) * add history request * Browse commit history of repo, directories, and files * Move routes to own pod (#2317) * Use https url for login (#2327) Removes a deprecation warning for SFAuthentication because of using http * Expand push settings info accessibility label (#2326) * cleans up issue labels query (#2334) * Move dropdown control to own lib (#2335) * Move dropdown control to own lib * update to add missing assets * fix interaction, set appearances * clear push notifications on open (#2336) * update dropdown view for centered title view (#2338) * inset preview collection view (#2340) * update routing lib and wire up new repo route (#2344) * Update reivewGitHUbAccessBtn constraints (#2339) * [ADD] "Try Beta" cell on Settings (#2346) Adds "Try Beta" cell on Settings screen, On tap opens the testflight invite link in safari * Message on "Try Beta" when already in TF (#2351) * [ADD] On Tap "Try Beta", show Squawk message when user is already on beta * Update Squawk+GitHawk.swift * Fixes regex for consecutive shortlinks (#2358) - updated unit tests * Fix typo in TF squawk (#2361) * Lowercase issue/PR search string (#2360) * fix lints * lowercase search string * fix more lints * Allow showing/cloning repository in Working Copy when installed (#2366) .../more button when looking at repository has extra action when Working Copy is installed to show or clone this repository in this app. * Fixes Reaction emoji updates on split view (#2359) * [FIX] Reaction emoji updates on split view * [FIX] Change listener to delegate * Update MessageViewController (#2370) * Update routes to final form (#2374) * update routes to final form * fix routes build and update with router object
1 parent 398edf2 commit 1c40d45

File tree

843 files changed

+22032
-55385
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

843 files changed

+22032
-55385
lines changed

.github/ISSUE_TEMPLATE/bug_report.md

Lines changed: 35 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,35 @@
1+
---
2+
name: Bug report
3+
about: Create a report to help us improve
4+
5+
---
6+
7+
**Describe the bug**
8+
A clear and concise description of what the bug is.
9+
10+
**To Reproduce**
11+
Steps to reproduce the behavior:
12+
1. Go to '...'
13+
2. Click on '....'
14+
3. Scroll down to '....'
15+
4. See error
16+
17+
**Expected behavior**
18+
A clear and concise description of what you expected to happen.
19+
20+
**Screenshots**
21+
If applicable, add screenshots to help explain your problem.
22+
23+
**Desktop (please complete the following information):**
24+
- OS: [e.g. iOS]
25+
- Browser [e.g. chrome, safari]
26+
- Version [e.g. 22]
27+
28+
**Smartphone (please complete the following information):**
29+
- Device: [e.g. iPhone6]
30+
- OS: [e.g. iOS8.1]
31+
- Browser [e.g. stock browser, safari]
32+
- Version [e.g. 22]
33+
34+
**Additional context**
35+
Add any other context about the problem here.
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
---
2+
name: Feature request
3+
about: Suggest an idea for this project
4+
5+
---
6+
7+
**Is your feature request related to a problem? Please describe.**
8+
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
9+
10+
**Describe the solution you'd like**
11+
A clear and concise description of what you want to happen.
12+
13+
**Describe alternatives you've considered**
14+
A clear and concise description of any alternative solutions or features you've considered.
15+
16+
**Additional context**
17+
Add any other context or screenshots about the feature request here.

.swiftlint.yml

Lines changed: 9 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,21 +1,24 @@
11
disabled_rules:
22
- function_parameter_count
3-
- line_length
4-
- identifier_name
5-
- type_name
63
- function_body_length
74
- type_body_length
85
- force_try
96
- force_cast
10-
- trailing_whitespace
7+
- cyclomatic_complexity
8+
- identifier_name
9+
- type_name
10+
- line_length
11+
- file_length
12+
- large_tuple
13+
- nesting
1114

1215
included:
1316
- Classes
1417
- FreetimeTests
18+
- Local\ Pods
1519
excluded:
1620
- Pods
17-
- Local Pods
1821
- Playgrounds
19-
- Sample
22+
- Local\ Pods/SwipeCellKit
2023

2124
reporter: "xcode"

AppStore.md

Lines changed: 0 additions & 110 deletions
This file was deleted.

CODE_OF_CONDUCT.md

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
# Contributor Covenant Code of Conduct
2+
3+
## Our Pledge
4+
5+
In the interest of fostering an open and welcoming environment, we as contributors and maintainers pledge to making participation in our project and our community a harassment-free experience for everyone, regardless of age, body size, disability, ethnicity, gender identity and expression, level of experience, nationality, personal appearance, race, religion, or sexual identity and orientation.
6+
7+
## Our Standards
8+
9+
Examples of behavior that contributes to creating a positive environment include:
10+
11+
* Using welcoming and inclusive language
12+
* Being respectful of differing viewpoints and experiences
13+
* Gracefully accepting constructive criticism
14+
* Focusing on what is best for the community
15+
* Showing empathy towards other community members
16+
17+
Examples of unacceptable behavior by participants include:
18+
19+
* The use of sexualized language or imagery and unwelcome sexual attention or advances
20+
* Trolling, insulting/derogatory comments, and personal or political attacks
21+
* Public or private harassment
22+
* Publishing others' private information, such as a physical or electronic address, without explicit permission
23+
* Other conduct which could reasonably be considered inappropriate in a professional setting
24+
25+
## Our Responsibilities
26+
27+
Project maintainers are responsible for clarifying the standards of acceptable behavior and are expected to take appropriate and fair corrective action in response to any instances of unacceptable behavior.
28+
29+
Project maintainers have the right and responsibility to remove, edit, or reject comments, commits, code, wiki edits, issues, and other contributions that are not aligned to this Code of Conduct, or to ban temporarily or permanently any contributor for other behaviors that they deem inappropriate, threatening, offensive, or harmful.
30+
31+
## Scope
32+
33+
This Code of Conduct applies both within project spaces and in public spaces when an individual is representing the project or its community. Examples of representing a project or community include using an official project e-mail address, posting via an official social media account, or acting as an appointed representative at an online or offline event. Representation of a project may be further defined and clarified by project maintainers.
34+
35+
## Enforcement
36+
37+
Instances of abusive, harassing, or otherwise unacceptable behavior may be reported by contacting the project team at [email protected]. The project team will review and investigate all complaints, and will respond in a way that it deems appropriate to the circumstances. The project team is obligated to maintain confidentiality with regard to the reporter of an incident. Further details of specific enforcement policies may be posted separately.
38+
39+
Project maintainers who do not follow or enforce the Code of Conduct in good faith may face temporary or permanent repercussions as determined by other members of the project's leadership.
40+
41+
## Attribution
42+
43+
This Code of Conduct is adapted from the [Contributor Covenant][homepage], version 1.4, available at [http://contributor-covenant.org/version/1/4][version]
44+
45+
[homepage]: http://contributor-covenant.org
46+
[version]: http://contributor-covenant.org/version/1/4/

Classes/Bookmark/BookmarkCell.swift

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ final class BookmarkCell: SwipeSelectableCell {
2525

2626
override init(frame: CGRect) {
2727
super.init(frame: frame)
28-
28+
2929
backgroundColor = .white
3030

3131
contentView.clipsToBounds = true
@@ -84,14 +84,14 @@ final class BookmarkCell: SwipeSelectableCell {
8484
string: "\(viewModel.bookmark.owner)/\(viewModel.bookmark.name)",
8585
attributes: [
8686
.font: Styles.Text.secondaryBold.preferredFont,
87-
.foregroundColor: Styles.Colors.Gray.light.color,
87+
.foregroundColor: Styles.Colors.Gray.light.color
8888
]
8989
)
9090
detailString.append(NSAttributedString(
9191
string: " #\(viewModel.bookmark.number)",
9292
attributes: [
9393
.font: Styles.Text.secondary.preferredFont,
94-
.foregroundColor: Styles.Colors.Gray.light.color,
94+
.foregroundColor: Styles.Colors.Gray.light.color
9595
]
9696
))
9797
detailLabel.attributedText = detailString

Classes/Bookmark/BookmarkNavigationController.swift

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ final class BookmarkNavigationController {
1414

1515
private let store: BookmarkStore
1616
private let model: Bookmark
17+
private static let iconImageInset = UIEdgeInsets(top: 0, left: 16, bottom: 0, right: 0)
1718

1819
init?(store: BookmarkStore?, model: Bookmark?) {
1920
guard let store = store, let model = model else { return nil }
@@ -22,19 +23,19 @@ final class BookmarkNavigationController {
2223
}
2324

2425
// MARK: Public API
25-
26+
2627
var navigationItem: UIBarButtonItem {
2728
let item = UIBarButtonItem()
2829
configureNavigationItem(item)
2930
return item
3031
}
31-
32+
3233
func configureNavigationItem(_ item: UIBarButtonItem) {
33-
34+
3435
let accessibilityLabel: String
3536
let imageName: String
3637
let selector: Selector
37-
38+
3839
if store.contains(model) {
3940
imageName = "nav-bookmark-selected"
4041
accessibilityLabel = Constants.Strings.removeBookmark
@@ -44,36 +45,38 @@ final class BookmarkNavigationController {
4445
accessibilityLabel = Constants.Strings.bookmark
4546
selector = #selector(BookmarkNavigationController.add(sender:))
4647
}
47-
48+
4849
item.accessibilityLabel = accessibilityLabel
4950
item.image = UIImage(named: imageName)?.withRenderingMode(.alwaysTemplate)
5051
item.target = self
5152
item.action = selector
5253
item.isEnabled = true
5354
item.width = 0
55+
item.imageInsets = BookmarkNavigationController.iconImageInset
5456
}
55-
57+
5658
//for timeframe between viewDidLoad() and bookmark info is loaded
5759
static var disabledNavigationItem: UIBarButtonItem {
5860
let item = UIBarButtonItem()
5961
item.image = UIImage(named: "nav-bookmark")?.withRenderingMode(.alwaysTemplate)
6062
item.isEnabled = false
63+
item.imageInsets = BookmarkNavigationController.iconImageInset
6164
return item
6265
}
63-
66+
6467
// MARK: Private API
65-
68+
6669
@objc func add(sender: UIBarButtonItem) {
6770
Haptic.triggerSelection()
6871
sender.action = #selector(BookmarkNavigationController.remove(sender:))
6972
sender.image = UIImage(named: "nav-bookmark-selected")?.withRenderingMode(.alwaysTemplate)
7073
store.add(model)
7174
}
72-
75+
7376
@objc func remove(sender: UIBarButtonItem) {
7477
sender.action = #selector(BookmarkNavigationController.add(sender:))
7578
sender.image = UIImage(named: "nav-bookmark")?.withRenderingMode(.alwaysTemplate)
7679
store.remove(model)
7780
}
78-
81+
7982
}

Classes/Bookmark/BookmarkSectionController.swift

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ final class BookmarkSectionController: ListGenericSectionController<BookmarkView
3232

3333
return CGSize(
3434
width: width,
35-
height: max(object.text.viewSize(in: width).height, Styles.Sizes.tableCellHeightLarge)
35+
height: max(object.text.viewSize(in: width).height, Styles.Sizes.tableCellHeight + Styles.Sizes.rowSpacing * 2)
3636
)
3737
}
3838

Classes/Bookmark/BookmarkViewController.swift

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@ TabNavRootViewControllerType {
7676

7777
searchBar.delegate = self
7878
searchBar.placeholder = Constants.Strings.searchBookmarks
79-
searchBar.tintColor = Styles.Colors.Blue.medium.color
8079
searchBar.backgroundColor = .clear
8180
searchBar.searchBarStyle = .minimal
8281
navigationItem.titleView = searchBar
@@ -99,12 +98,11 @@ TabNavRootViewControllerType {
9998
collectionView.collectionViewLayout.invalidateForOrientationChange()
10099
}
101100
}
102-
101+
103102
override func viewWillDisappear(_ animated: Bool) {
104103
searchBar.resignFirstResponder()
105104
}
106105

107-
108106
private func update(animated: Bool) {
109107
adapter.performUpdates(animated: animated)
110108
}

0 commit comments

Comments
 (0)