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

Adjusted changelog and version to 0.2.5 #2325

Merged
merged 1 commit into from
Oct 25, 2019
Merged

Adjusted changelog and version to 0.2.5 #2325

merged 1 commit into from
Oct 25, 2019

Conversation

LukasHirt
Copy link
Contributor

No description provided.

@LukasHirt LukasHirt self-assigned this Oct 25, 2019
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated
- Rename and delete will be disallowed in case the parent folder has no permissions fot these two operations https://github.com/owncloud/phoenix/pull/2129
- Acceptance tests for deleting shared files and deleting shares https://github.com/owncloud/phoenix/pull/2140
- Reject promise if invalid response during permalink check in acceptance tests https://github.com/owncloud/phoenix/pull/2141
- Acceptance test for checking delete button state when prohibiting deleting in share https://github.com/owncloud/phoenix/pull/2146
Copy link
Contributor

Choose a reason for hiding this comment

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

note that for core I usually would not add any entries that only add tests, not sure if we should that the same here

@LukasHirt LukasHirt force-pushed the release/25102019 branch 2 times, most recently from 41fc10d to 62775d4 Compare October 25, 2019 14:27
CHANGELOG.md Outdated
### Added
- Run tests on internet explorer https://github.com/owncloud/phoenix/pull/2082
- Draw.io app integration https://github.com/owncloud/phoenix/pull/2083
- Logout step for acceptance tests https://github.com/owncloud/phoenix/pull/2100
Copy link
Contributor

Choose a reason for hiding this comment

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

also remove test entries such as these ?

Moved few PRs to different sections

Removed tests from changelog

Removed last tests items

Changed wording of ie 11 tests
Copy link
Contributor

@PVince81 PVince81 left a comment

Choose a reason for hiding this comment

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

👍

@PVince81 PVince81 merged commit 65599b9 into master Oct 25, 2019
@delete-merged-branch delete-merged-branch bot deleted the release/25102019 branch October 25, 2019 14:56
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.

2 participants