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

Issue #62: Chat message send icon was not refreshed #7847

Closed
wants to merge 210 commits into from

Conversation

vickyokrm
Copy link
Contributor

@vickyokrm vickyokrm commented Aug 23, 2017

No description provided.

mrsimpson and others added 30 commits February 14, 2017 13:50
…, remove hack which existed for node 0.10.xx
No migration on DB implemented, has to be done manually
Not beautified yet and not yet visible in the lefthandside navbar.
mrsimpson and others added 27 commits July 17, 2017 08:19
* Allow lowercase topic names
* Error handling Smarti not available : Do not create `undefined` script tags
* Help-request actions button shall disappear once the help-request has been closed (#38)
* Make requests for expertises respect case as well (#36)
* Remove subscription of owner who created a request on closing a help request (fixes #38)
* #47 - Closing a help request does not store the conversation correctly
- Added some debugging output
- Some formatting
* Do not lowercase experts channel
* Satisfy linter and remove console dependency
Fixes a small / not related lint Issue
add configuration parameter for Assistify Widgets posting behaviour
Conflicts:
Expertise and request room types to be considered in files which have been converted from .coffee to .js:

- packages/rocketchat-ui/client/views/app/room.js
- packages/rocketchat-ui-admin/client/rooms/adminRooms.js

package.js merged
Conflicts:
Expertise and request room types to be considered in files which have been converted from .coffee to .js:

- packages/rocketchat-ui/client/views/app/room.js
- packages/rocketchat-ui-admin/client/rooms/adminRooms.js

package.js merged
…x` has done to the file previously...

Note to myself: report error in `stylelint --fix`
The actual correction would be to provide a real authorization object, but that lead to exceptions which could not be corrected so far.
Thus, this commit provides a short-term-low-risk-workaround.
@CLAassistant
Copy link

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 5 committers have signed the CLA.

✅ mrsimpson
✅ ruKurz
github@beimir.net
❌ Oliver Jägle
❌ tkurz


github@beimir.net, Oliver Jägle seem 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.

@vickyokrm vickyokrm closed this Aug 23, 2017
@vickyokrm
Copy link
Contributor Author

Pls Ignore this pull request

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.

4 participants