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.3] Milestone 12.0.3 #1236

Merged
merged 61 commits into from
Sep 5, 2023
Merged

[milestone/12.0.3] Milestone 12.0.3 #1236

merged 61 commits into from
Sep 5, 2023

Conversation

felix-schwarz
Copy link
Contributor

@felix-schwarz felix-schwarz commented Jun 28, 2023

Description

Implements these changes:

  • new: adds mtime metadata to TUS uploads
  • new: CSS gains variable support (via experimental/branding)
  • improved: App Provider fixes / feature detection
  • fix: update connection name in sidebar after it has been edited
  • fix: avoid unwanted Auto Layout collapse of select button in location picker / BottomButtonBar
  • fix: FileProvider slowness and hangs
  • fix: FileProvider crash when being passed a nil item identifier by iOS
  • updated translations via Translation sync #1269

Related Issues

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

…supportsCreateDocument

- AccountConnection: create Open with actions dependant on OCAppProvider.supportsCreateDocument
- OpenInWebAppAction:
	- introduce new "auto" mode that uses the best available method - or shows an error
	- limit size of icons to a maximum of 32x32 points
- update SDK for new feature detection capabilities
@CLAassistant
Copy link

CLAassistant commented Jun 28, 2023

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.
2 out of 4 committers have signed the CLA.

✅ hosy
✅ michaelstingl
❌ felix-schwarz
❌ Michael Stingl


Michael Stingl seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

felix-schwarz and others added 28 commits July 3, 2023 12:33
- add missing errorHandler calls to FileProviderContentEnumerator
- add last resort enumeration completion call to FileProviderEnumeratorObserver.dealloc
- add additional debug logging
Co-authored-by: Michael Stingl <[email protected]>
* Remove old PLIST examples

* move ownCloud.online theme + assets

* move EMM assets

* Adjust Fastfile for moved themes + assets

* updated to Xcode 14.3

* move all app resources to the correct place in one copy command

* updated to 14.3.1

---------

Co-authored-by: Michael Stingl <[email protected]>
Co-authored-by: Matthias Hühne <[email protected]>
- added accessibility label to the logo image
- add support for RGBA hex values to CSS
…screen background (#1264)

* - add background image to welcome screen
- add support for RGBA hex values to CSS

* Assets.xcassets/AppIcon.appiconset generated with fastlane

* align icon name with branding

---------

Co-authored-by: Felix Schwarz <[email protected]>
Co-authored-by: Michael Stingl <[email protected]>
…cords for colors, incl. a set of system colors

- ThemeCollection: add system colors (f.ex. UIColor.systemBlue as `os.color.blue`) and theme base colors (as `theme.color.light` and `theme.color.dark`) to CSS object
	- add permanent logo branding to top of the view
	- make branding elements use same CSS selectors as welcome screen
- ComposedMessageView: make CSS selectors extension public
- RoundCornerBackgroundView: add new .fillImage property that allows filling the view with an image with "scaleAspectFill" contentMode
@michaelstingl
Copy link
Contributor

In addition to the comments above, the login view does not look like nice in iOS17. Fix here or in following releases?

Uh. not nice. Worse then I expected. I thought color-wise, everything is better in iOS 17… 🤔

@jesmrec
Copy link
Contributor

jesmrec commented Aug 28, 2023

@felix-schwarz was this fix included in the release branch? It is closed but i can reproduce the problem in b5391b8 . Changes in the name are not applied till apps is reopened.

@felix-schwarz
Copy link
Contributor Author

@felix-schwarz was this fix included in the release branch? It is closed but i can reproduce the problem in b5391b8 . Changes in the name are not applied till apps is reopened.

It was merged into milestone/12.0.3 and closed after @michaelstingl could no longer reproduce the issue after that. I also just tried to reproduce the issue with the latest build but couldn't. Can you share a video?

…s that are also subject to NSObject.applyThemeCollection() calls
…tup view into own view controller

- InitialSetupViewController: implement status bar style support for consistent status bar colors also pre-iOS 17
@felix-schwarz
Copy link
Contributor Author

felix-schwarz commented Aug 28, 2023

@jesmrec @michaelstingl All color issues (including status bar and iOS 17) should be fixed as of the latest commit / build 280.

@michaelstingl
Copy link
Contributor

Yeah, rock-solid in iOS 16 + 17, following system dark + light mode ✅

@jesmrec
Copy link
Contributor

jesmrec commented Aug 29, 2023

It was merged into milestone/12.0.3 and closed after @michaelstingl could no longer reproduce the issue after that. I also just tried to reproduce the issue with the latest build but couldn't. Can you share a video?

@felix-schwarz here it is:

RPReplay_Final1693287049.MP4

@jesmrec
Copy link
Contributor

jesmrec commented Aug 29, 2023

@jesmrec @michaelstingl All color issues (including status bar and iOS 17) should be fixed as of the latest commit / build 280.

fixed in a455ce1dc.

Still dark login view, but it will be expected in 12.0.3

@felix-schwarz
Copy link
Contributor Author

@jesmrec Thanks for the video. I can now reproduce it on an iPhone when the bookmark did not have a name previously. Looking into it.

@jesmrec
Copy link
Contributor

jesmrec commented Aug 29, 2023

@felix-schwarz also, detected this glitch in an iPhoneX. I guess it's caused because the screen width:

Settings > Media Upload > Auto Upload Photos -> switch on

Button to select destination is not correct. Curiously, just happens with Auto Upload Photos and not with Videos

Screenshot 2023-08-29 at 10 57 18

iPhoneX 16.5
a455ce1d

@felix-schwarz
Copy link
Contributor Author

@jesmrec The rename issue is fixed as of 1932993.

…e promptText above the buttons in any but regular horizontal class size and stretching the buttons to fill the space beneath it
@felix-schwarz
Copy link
Contributor Author

felix-schwarz commented Aug 29, 2023

@jesmrec The layout glitch is also fixed now. The button bar layout now adjusts to the horizontal size class and puts the prompt text above the buttons if the size class is not regular:

Bildschirmfoto 2023-08-29 um 15 04 18

@jesmrec
Copy link
Contributor

jesmrec commented Aug 29, 2023

Everything fixed

@jesmrec
Copy link
Contributor

jesmrec commented Aug 29, 2023

QA

This is a bugfixing release, so no complete regression test is required

Changelog

Smoke test

Server version: 10.13.0 && 4.0.0
Device & iOS version: iPhoneX, iOS 16.5.1 (REAL), iPhone 14, iOS 16.5 (SIM), iPhoneXR 17.0 (REAL)

Test Case Steps Expected Result Result Related Comment
Login Basic Login in an oC10 server, typing correct credentials Access to account Passed AUTO
New folder Open '+' and select to create a new folder with correct name The folder is created with the correct name Passed AUTO
Rename 1. Select an item by clicking on 3-dot-button
2. Select Rename and type new correct name
Item is renamed with the new name Passed AUTO
Move 1. Select an item by clicking on 3-dot-button
2. Select Move, and a new location
Folder is moved to the specified location Passed AUTO
Copy 1. Select a folder by clicking on 3-dot-button
2. Select Copy, and a location to copy to (including space)
The folder and all its content is copied to the location, and also remains in the original Passed AUTO
Duplicate 1. Select a folder by clicking on 3-dot-button
2. Select Duplicate
The folder and all its content is duplicated in the current location, with the name adding 2 Passed AUTO
Delete 1. Select a folder by clicking on 3-dot-button
2. Select Delete and submit deletion
Item deleted Passed AUTO
Upload Files Select many files from File apps to upload to oC Files are uploaded Passed AUTO
Upload Albums Select 300 images and videos from Albums to upload to oC Images and videos are uploaded Passed
Download Tap to download a file The file is downloaded and previewed Passed
Av Offline 1. Select a folder by clicking on 3-dot-button
2. Select set as Av Offline file
The file is downloaded and set as Av Offline Passed
Av Offline folder 1. Select a folder with subfolders inside by clicking on 3-dot-button
2. Select Av. offline
3. Add new content to the folder
2. The folder and all its content is downloaded and set as Av Offline
3. New content is available offline
Passed
Share with user 1. Select file and folder by clicking on 3-dot-button
2. Select Share and add new recipient
3. Recipient accepts the share
Both are correctly shared and visible by the recipient in Shares space Passed AUTO
Link Create a link over a file and a folder Links are correctly created Passed AUTO
Spaces 1. In web, create some spaces that are visible for the user List of spaces is visible in the account Passed AUTO
Files App 1. Add several accounts
2. Open Files app
All accounts are correctly listed and are browsable Passed

Autom test report vs oCIS

Autom test report vs oC10

Upgrade test

12.0.2 -> 12.0.3 with oC10 (basic), oC10 (OAuth2), oCIS (OIDC) ✅


Approved on my side.

Approved commits:

app: 40dfc92
sdk: 740b63e

Devices:

iPhoneX 16.6
iPhoneXR 17.0 (just basic checks)

Xcode version: 14.3.1 (14E300c)

@hosy hosy merged commit 1107434 into master Sep 5, 2023
@delete-merged-branch delete-merged-branch bot deleted the milestone/12.0.3 branch September 5, 2023 11:49
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.

5 participants