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

Remove obsolete contacts_cards_properties table #157

Merged
merged 1 commit into from
Jun 20, 2016

Conversation

nblock
Copy link
Contributor

@nblock nblock commented Jun 19, 2016

The table contacts_cards_properties was part of the contacts app until
ownCloud 8.2. It got replaced with cards_properties (part of dav).

See also: owncloud/core#21889

The table contacts_cards_properties was part of the contacts app until
ownCloud 8.2. It got replaced with cards_properties (part of dav).

See also: ownCloud/core nextcloud#21889
@MariusBluem
Copy link
Member

Why was this not already done with owncloud/core#24166 😕

cc @MorrisJobke (because you participated the pull above 😅)

@MariusBluem MariusBluem added the 3. to review Waiting for reviews label Jun 19, 2016
@MariusBluem MariusBluem added this to the Nextcloud 10 milestone Jun 19, 2016
@MorrisJobke
Copy link
Member

Makes sense 👍

@MorrisJobke
Copy link
Member

Why was this not already done with owncloud/core#24166 😕

I have no idea 🙈

@MariusBluem
Copy link
Member

MariusBluem commented Jun 19, 2016

👍
Why is continuous-integration/drone not running 😱 @MorrisJobke

@MorrisJobke
Copy link
Member

Why is continuous-integration/drone not running 😱 @MorrisJobke

Ummm ... no idea ...

@MorrisJobke
Copy link
Member

It somehow failed to deliver. I redeliverd it and it runs now.

bildschirmfoto 2016-06-20 um 10 59 21

@MorrisJobke MorrisJobke added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Jun 20, 2016
@MariusBluem MariusBluem merged commit fffe1be into nextcloud:master Jun 20, 2016
@MariusBluem
Copy link
Member

THX.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4. to release Ready to be released and/or waiting for tests to finish
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants