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

Talk #444

Merged
merged 8 commits into from
Oct 26, 2017
Merged

Talk #444

merged 8 commits into from
Oct 26, 2017

Conversation

nickvergessen
Copy link
Member

No description provided.

@nickvergessen nickvergessen added this to the 3.0 (Nextcloud 13) milestone Oct 19, 2017
Signed-off-by: Joas Schilling <[email protected]>
@jancborchardt
Copy link
Member

(Btw, while we’re renaming the app, can we also use this opportunity to rename the repository? Currently it’s very confusing for anyone wanting to open issues or contribute, and we could use this break to fix that. @karlitschek @nickvergessen)

@nickvergessen
Copy link
Member Author

We currently decided against changing the app id, because nextcloud can not handle this gracefully. So we would need to make an update for spreed telling people to install talk, etc.
this is way to confusing and there we decided to stick with it on the id level and only change the UI.

Copy link
Member

@Ivansss Ivansss left a comment

Choose a reason for hiding this comment

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

Notifications are not visible anymore.

@nickvergessen
Copy link
Member Author

Fixed

Signed-off-by: Joas Schilling <[email protected]>
@Ivansss Ivansss merged commit 9928c4d into master Oct 26, 2017
@Ivansss Ivansss deleted the talk branch October 26, 2017 10:13
marcoambrosini pushed a commit that referenced this pull request Oct 9, 2019
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.

3 participants