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

[milestone/12.0] Version 12.0 #1095

Merged
merged 374 commits into from
Jun 14, 2023
Merged

[milestone/12.0] Version 12.0 #1095

merged 374 commits into from
Jun 14, 2023

Conversation

felix-schwarz
Copy link
Contributor

@felix-schwarz felix-schwarz commented Jan 24, 2022

Description

PR for version 12.0:

Related Issue

https://github.com/owncloud/ios-app/issues/1221)](https://github.com/owncloud/ios-app/issues/1221

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • Added an issue with details about all relevant changes in the iOS documentation repository.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • Added changelog files for the fixed issues in folder changelog/unreleased

QA

Test plans:

Regression: https://github.com/owncloud/QA/blob/master/Mobile/iOS-app/Version%2012.0/Regression.md
Spaces: https://github.com/owncloud/QA/blob/master/Mobile/iOS-app/Version%2012.0/1101_spaces.md
Sharing in sidebar: https://github.com/owncloud/QA/blob/master/Mobile/iOS-app/Version%2012.0/Sharing-oCIS.md

(need merging)

Reports:

@CLAassistant
Copy link

CLAassistant commented Jan 24, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ hosy
❌ felix-schwarz
You have signed the CLA already but the status is still pending? Let us recheck it.

@felix-schwarz felix-schwarz added this to the 12.0.0-Next milestone Jan 24, 2022
@felix-schwarz felix-schwarz changed the base branch from master to feature/avatar January 24, 2022 21:15
@felix-schwarz felix-schwarz changed the base branch from feature/avatar to master June 30, 2022 10:56
- address open/public Swift compiler warnings
# Conflicts:
#	CHANGELOG.md
#	ios-sdk
#	ownCloud.xcodeproj/project.pbxproj
#	ownCloud/Client/Actions/EditDocumentViewController.swift
#	ownCloud/Client/Viewer/DisplayViewController.swift
#	ownCloud/Resources/de.lproj/Localizable.strings
#	ownCloudAppShared/Client/File Lists/QueryFileListTableViewController.swift
#	ownCloudAppShared/User Interface/Theme/NSObject+ThemeApplication.swift
# Conflicts:
#	ios-sdk
#	ownCloud.xcodeproj/xcshareddata/xcschemes/ownCloud.xcscheme
#	ownCloud/Client/Actions/Actions+Extensions/OpenInAction.swift
#	ownCloud/Client/Viewer/DisplayViewController.swift
- respect OCCapabilities.federatedSharingSupported
- GroupSharingTableViewController: fix delete share issue by calling the correct method
- adapt to OCShare.itemType type change
- replace UIImage(systemName:) code with one using OCSymbol
- fix more Xcode 14 warnings
- ClientItemViewController:
	- implement support for "folder removed" state
	- adjust message spacing
	- refactor navigation bar code
	- add footer to display statistics for the current location, feeding from OCQuery-provided OCStatistics
	- prepare to also include quota information in the footer
- update KNOWN_ISSUES.md
- ViewCell: add separatorLayoutGuideCustomizer to allow customization of separator layout guide for individual views
- ClientItemViewController: add support to display quota information (space remaining)
- UIView+EmbedAndLayout: add code to embed multiple views vertically, modify existing ConstraintSet type
…into themselves

- ViewCell: fix seperatorLayoutGuide constraint on reuse (could cause visual glitch of full-length separator line when there shouldn't have been one)
- update known issues
…pp presentation

- ClientWebAppViewController: implement logic to immediately dismiss the view controller when tapping the close button, but keep the web view alive until the "close" event arrives from the webview - or 10 seconds have passed
- OpenInAction: honor excluded activities, bring up UIActivityViewController instead of UIDocumentInteractionController even for a single item if activities are excluded
- ownCloud.xcscheme: add env var example for action.excludedSystemActivities to block copying to pasteboard
…nnect() to enrich bookmark with metadata and user name during setup. Required to present the correct UI for space-based accounts.
felix-schwarz added a commit that referenced this pull request Jun 13, 2023
… for laying out the view in a keyboard-responsive manner

- CollectionViewController: add .compressForKeyboard property that layouts the UICollectionView in a keyboard-responsive manner
- ShareViewController: adopt compressForKeyboard
- ClientItemViewController: adopt compressForKeyboard
- fixes finding (1) in #1095
@felix-schwarz
Copy link
Contributor Author

@jesmrec #1220 fixes (1) and (7)

@felix-schwarz felix-schwarz requested review from hosy and jesmrec June 14, 2023 07:15
felix-schwarz added a commit that referenced this pull request Jun 14, 2023
@felix-schwarz
Copy link
Contributor Author

@jesmrec #1220 now also fixes (9)

felix-schwarz added a commit that referenced this pull request Jun 14, 2023
@felix-schwarz
Copy link
Contributor Author

@jesmrec #1220 now also fixes (12). That string is stored in the File Provider UI's Info.plist and therefore can't be changed dynamically at runtime. To keep it branding-friendly, I chose "Share by cloud" instead of "Share via ownCloud", "Share (ownCloud)", which would have required to customize both the Info.plist and the translations for branding.

felix-schwarz added a commit that referenced this pull request Jun 14, 2023
- MediaDisplayViewController: when streaming, show a loading indicator (fixing finding (5) in #1095)
@felix-schwarz
Copy link
Contributor Author

@jesmrec finding (5) is also fixed in #1220 now

@hosy hosy merged commit 8f6b87d into master Jun 14, 2023
@delete-merged-branch delete-merged-branch bot deleted the milestone/12.0 branch June 14, 2023 13:23
@felix-schwarz felix-schwarz restored the milestone/12.0 branch June 14, 2023 19:04
felix-schwarz added a commit that referenced this pull request Jun 14, 2023
… for laying out the view in a keyboard-responsive manner

- CollectionViewController: add .compressForKeyboard property that layouts the UICollectionView in a keyboard-responsive manner
- ShareViewController: adopt compressForKeyboard
- ClientItemViewController: adopt compressForKeyboard
- fixes finding (1) in #1095
felix-schwarz added a commit that referenced this pull request Jun 14, 2023
felix-schwarz added a commit that referenced this pull request Jun 14, 2023
felix-schwarz added a commit that referenced this pull request Jun 14, 2023
- MediaDisplayViewController: when streaming, show a loading indicator (fixing finding (5) in #1095)
felix-schwarz added a commit that referenced this pull request Jun 14, 2023
…g video files (fixing part 2 of finding (5) in #1095)
hosy added a commit that referenced this pull request Jun 20, 2023
* - BrowserNavigationViewController:
	- add ability to show/hide navigation bar (with or without animation)
	- add convenience accessor to UIViewController to access the enclosing browserNavigationViewController
- Branding+App: fix SwiftLint warnings
- PDFViewerViewController: hide navigation bar via browserNavigationController rather than navigationController (fix finding (7))

* - UIView+EmbedLayout: add new .safeAreaWithKeyboardAnchorSet property for laying out the view in a keyboard-responsive manner
- CollectionViewController: add .compressForKeyboard property that layouts the UICollectionView in a keyboard-responsive manner
- ShareViewController: adopt compressForKeyboard
- ClientItemViewController: adopt compressForKeyboard
- fixes finding (1) in #1095

* - fix finding (9) in #1095

* - fix finding (12) in #1095

* - ComposedMessageView: apply syntactic sugar
- MediaDisplayViewController: when streaming, show a loading indicator (fixing finding (5) in #1095)

* - fix localization typo

* - MediaDisplayViewController: show download indicator when downloading video files (fixing part 2 of finding (5) in #1095)

* - BottomButtonBar: vertically align also the prompt label to selectButton.centerYAnchor
- ClientLocationPicker: prevent usage of .keyboardLayoutGuide to fix finding (13) in #1220 (misplaced select button after navigating to Files view)
- CollectionViewController: use the .safeAreaWithKeyboardAnchorSet for the whole stack view rather than just the collection view

* - new build and version number
- added in-app release notes
- added calens release notes

* Calens changelog updated

* updated fastlane release notes

---------

Co-authored-by: Matthias Hühne <github@hosy.de>
Co-authored-by: hosy <hosy@users.noreply.github.com>
Co-authored-by: Matthias Hühne <mhuehne@owncloud.com>
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.

6 participants