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

Apply new auth form design #846

Merged
merged 3 commits into from Sep 6, 2021
Merged

Apply new auth form design #846

merged 3 commits into from Sep 6, 2021

Conversation

JammingBen
Copy link
Contributor

@JammingBen JammingBen commented Sep 6, 2021

Which was introduced in OC 10.8.

Similar to owncloud/core#38894
Fixes owncloud/core#39103

image

@AlexAndBear AlexAndBear mentioned this pull request Sep 6, 2021
44 tasks
@phil-davis
Copy link
Contributor

Note: oC10 core master has dropped PHP 7.2. For CI we need to stop trying to run unit tests with PHP 7.2.

PR #845 drops all PHP 7.2 (including updating composer.json to PHP minimum 7.2 - we probably do not want to merge that right now, because ownCloud 10.8.0 still supports PHP 7.2, and we want the app release to support some install that is still on ownCloud 10.8.0 + PHP 7.2

I will push smaller CI changes here to get it green.

@phil-davis
Copy link
Contributor

I pushed the minimum change for CI. This looks good. IMO ready to call some developer for review.

@AlexAndBear AlexAndBear merged commit 203a3c5 into master Sep 6, 2021
@phil-davis phil-davis deleted the password-form-design branch September 6, 2021 12:39
@jnweiger jnweiger mentioned this pull request Oct 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unable to focus password input box on auth screen for a public link that is a protected share
3 participants