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

No require avatar fixes #147

Merged
merged 3 commits into from
Jun 4, 2015
Merged

No require avatar fixes #147

merged 3 commits into from
Jun 4, 2015

Conversation

rodrigok
Copy link
Member

@rodrigok rodrigok commented Jun 4, 2015

fixes #125

@rodrigok rodrigok changed the title No require avatar fixes #125 No require avatar fixes Jun 4, 2015
rodrigok added a commit that referenced this pull request Jun 4, 2015
@rodrigok rodrigok merged commit 94fbdf0 into master Jun 4, 2015
@rodrigok rodrigok deleted the no-require-avatar branch June 4, 2015 18:44
Peym4n pushed a commit to redlink-gmbh/Rocket.Chat that referenced this pull request Jan 17, 2018
* WIP: Proxy for Smarti stub

* WIP: Proxy for Smarti stub added to package

```
curl -X POST \
  http://localhost:3000/api/v1/assistify/smarti/conversation/ \
  -H 'content-type: application/json' \
  -H 'x-auth-token: EKZ2_t7Wfqg5GML8bs44v3uS0d_b7f3rUd9IUYKNpvJ' \  //from login
  -H 'x-user-id: S4Mew2PDtEPr3Ejne' \                               //from login
  -d '{
	"thisIsMy":"JSON-Body"
}'
```

* use injected logger for logging in API

* Fixes RocketChat#151 - Misspelled label "jetzt chaten"

* Using the configured access token as HTTP header

* Using the RC proxy for Smarti

* Revert "Feature/mrsimpson#23 title first message to new request (RocketChat#149)"

This reverts commit 484b04c.

* Revert "Fixes RocketChat#151 - Misspelled label "jetzt chaten""

This reverts commit 0c9ac4f.

* Fixing lint errors.

* Fixing non authorized access.

* fixed issue with wrong knowledge provider indexes

* improved tailing slashes for URLs

* - add authorization checks before routing the API calls
- roll back injected logger, since it is not defined

* removed unused localization keys

* Reducing code for adding a tailing slash to the Smarti URL.

* consolidate constants naming

* - Use RateLimiter for Smarti requests
- Use propagation function for 'onMessage' and 'onClose'
- Make Smarti the default knowledge provider
- Reordered Smarti backend settings
- Added descriptions for Smarti backend settings
- Also reload the settings, when reloading the Smarti widget
- Refactored file names

* Using rate limiter in each proxy method and limit the propagateToSmarti function instead of HTTP.call

* - merged proxy and adapter into only one file
- only use DDP for Smarti Widget / Rocket.Chat messages (getConversation, getSmartiQueryBuilderResult)
- added proxy endpoint for Smarti conversation search
- added several comments

* Migrate settings

* Extract Smarti loader

* additional migrations

* Do not migrate old settings with other defaults

* separated responsibilities

* Simplify adapter, proxy, router, widgetBackend

* Minor fixes to get it work with Smarti.

* Fix Webhook-token not being transmitted

* revert unintentional changes to other files after branching off the wron state.
Copied all changes from `develop` which were not included in `assistify:ai`-package
HappyTobi pushed a commit to HappyTobi/Rocket.Chat that referenced this pull request Jul 10, 2018
HappyTobi pushed a commit to HappyTobi/Rocket.Chat that referenced this pull request Jul 10, 2018
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.

1 participant