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

Fix avatar being invalid in the Contact API call #863

Merged
merged 2 commits into from Jan 28, 2018
Merged

Conversation

djaiss
Copy link
Member

@djaiss djaiss commented Jan 28, 2018

This will close monicahq/chandler#48

@codecov
Copy link

codecov bot commented Jan 28, 2018

Codecov Report

Merging #863 into master will increase coverage by 0.02%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #863      +/-   ##
==========================================
+ Coverage   27.69%   27.71%   +0.02%     
==========================================
  Files         173      173              
  Lines        4749     4749              
==========================================
+ Hits         1315     1316       +1     
+ Misses       3434     3433       -1
Impacted Files Coverage Δ
app/Http/Resources/Contact/Contact.php 0% <0%> (ø) ⬆️
app/Contact.php 60.13% <100%> (ø) ⬆️
app/Reminder.php 100% <0%> (+3.84%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e6f8413...d2c0d5d. Read the comment docs.

@djaiss djaiss merged commit 1944190 into master Jan 28, 2018
@djaiss djaiss deleted the fix-api-avatar branch February 11, 2018 16:27
turtles2 added a commit to turtles2/monica that referenced this pull request Apr 1, 2018
* Add translation for preferences updated successfully message (monicahq#797)

* Add ability to export a contact as CSV (monicahq#840)

* Fix contact field types not being linked to account during vcard import (monicahq#841)

Fix monicahq#838

* Add more unit test coverage for money helper (monicahq#842)

* Add more unit tests for Gift model (monicahq#843)

* Add Code Coverage to Browser tests (monicahq#813)

* Using Live-code-coverage to get Code Coverage from http server
* Add a new test to pass Login form
* Add FakeUserTableSeeder to seed in testing env
* Using SESSION_DRIVER=file in travis env
* Add saucelabs connection (disable right now)
* Update composer

* Move codecov line (monicahq#844)

* Add more unit tests for User, Tag, Reminder and PetCategory models (monicahq#845)

* Add final unit tests for DateHelper (monicahq#846)

* Add more unit tests for Task, User and Tag models (monicahq#848)

* 2FA tests cover (monicahq#849)

* More unit tests for many models (monicahq#851)

* Add margin bottom to add contact form (monicahq#850)

* Do not put specific editor files in gitignore (monicahq#855)

* Bump version to 1.6.2 (monicahq#856)

* Some more 2fa test (monicahq#854)

* Fix avatar being invalid in the Contact API call (monicahq#863)

* Add Annual plan (monicahq#861)

* Fix annual plans generating an error for users exempt of the paid plan (monicahq#864)

* Fix a regression when filtering the list of contacts based on tags (monicahq#801)

* Update laravel-mix to version 2.0.0 (monicahq#847)

* Add requirements section in README (monicahq#858)

* Revert Laravel mix to 1.7.2 (monicahq#866)

* Remove env variable from prod (monicahq#867)

* Fix jquery-tags-input name import (monicahq#859)

* Set DB_PREFIX to a real empty string (monicahq#873)

* Fix Gitter link and readd badge for sonar (monicahq#871)

* Add gifts_update_success (monicahq#876)

* Update changelog with recent changes (monicahq#877)

* Test php 7.2 and postgresql (monicahq#785)

Running:
- php 7.1 + mysql + selenium (browser) tests
- php 7.2 + mysql
- php 7.1 + postgresql

* Add support for custom genders (monicahq#879)

* Refactor destroyAndReplace method to facilitate unit testing (monicahq#880)

* Add missing features in README (monicahq#881)

* Activate saucelabs connection (monicahq#882)

* Activate saucelabs (missing file) (monicahq#883)

See monicahq#882

* Build assets on travis and deploy them on bintray (monicahq#884)

* Fix deploy.json creation (monicahq#888)

* Fix docker image (monicahq#885)

* Remove nodejs and 'npm install' from image
* Remove everything that is not usefull
* Run composer with no-dev

* Don't build docker image immediatly (monicahq#889)

* Don't build docker image immediatly
* Fix php serve making too much output
* Print test results in script part

* Better edit contact screen (monicahq#890)

* Remove import contact function from Personalization screen (monicahq#891)

* Launch browser tests with Dusk (monicahq#892)

* And remove old (steward) tests

* Add ability to search contacts through the API (monicahq#895)

* Add default avatar color in the API (monicahq#896)

* Fix default genders seed with setup:production (monicahq#898)

* Fix heroku deploy (monicahq#901)

default login created after installation : 'admin@admin.com' / 'admin'

* Tiny typo fix (monicahq#900)

* Update installation docs (monicahq#903)

* Bump version to 1.7.0 (monicahq#908)

* Fix setup:production command (monicahq#909)

* Bump version to 1.7.1 (monicahq#910)

* Run tests with php 7.0 (monicahq#904)

* Update vagrant script and doc (monicahq#907)

* Remove trailing slash from post/get requests (monicahq#912)

* Cleanup continuous integration environment (monicahq#913)

Move scripts in a subdirectory

* Fix contact deletion not working (monicahq#915)

* Improve travis build (monicahq#916)

* Bump version 1.7.2 (monicahq#919)

* Docker run real cron (monicahq#906)

* Use crontab for schedules
* Update apache conf file
* Clean composer cache after installation
* Clean apk cache after installation

* Fix travis file for deploy stages (monicahq#926)

* Fix bintray tag set (monicahq#927)

* Fix Patreon link in README (monicahq#923)

* Remove trailing slash from routes (monicahq#918)

* Add ability to associate and remove one or more tags to a contact via the API (monicahq#924)

3 new API methods have been added:

* It allows to associate one or more tags with a contact,
* It allows to remove one or more tags from a contact,
* It allows to remove all the tags from a contact at once.

* Improve contact search through the API (monicahq#930)

* Add sort capabilities in the API (monicahq#933)

* Add concept of notifications for reminders (monicahq#894)

* Add crowdin configuration file (monicahq#936)

* Rename some i18n folders (monicahq#938)

* Update french translations (monicahq#939)

* Fix broken search (monicahq#941)

*  Fixes based on sonar recommandations (monicahq#934)

* Bump version 1.8.0 (monicahq#942)

* Fix bintray deploy (monicahq#943)

* Support unix socket to connect with mysql (monicahq#928)

* Remove saucelabs (monicahq#945)

* Test if .git directory is present (monicahq#950)

* Fix notifications not being sent (monicahq#951)

* Fix reset account deleting default account values (monicahq#956)

This fixes monicahq#922

* Remove partial contacts from search results returned by the API (monicahq#957)

* Fix gravatar support (monicahq#944)

* Fix messages and localization on contact edit (monicahq#955)

* Bump version to 1.8.1 (monicahq#962)

* Fix docker image creating wrong storage directories (monicahq#963)

* Update translations for it, pt, cz, ru, fr, de, cs (monicahq#940)

* Add bullet points to call lists when rendered from markdown (monicahq#961)

* Create a vagrant box (monicahq#967)

* Build vagrant box on travis (monicahq#968)

* Switching doc to the wiki (monicahq#969)

* Update SQL Import instructions link (monicahq#972)

* Fix issues found by static analysis (monicahq#878)

* sonar-launch file has moved (monicahq#974)

* Optimizations for composer (monicahq#952)

* Add apcu optimization
* Update composer

* Require debugbar on dev only (monicahq#983)

* Improve heroku integration (monicahq#970)

* Use google2fa-laravel dev-master (monicahq#949)

* Improve tests support (monicahq#984)

* Use WithFaker trait
* Add a scrollTo macro
* Use specific chromedriver

* Fix issues proposed by sonar (monicahq#986)

* Remove env calls (monicahq#991)

* Add codecov file (monicahq#993)

* Fix Faker trait use (monicahq#995)

* Update dependencies (monicahq#985)

* Update CONTRIBUTING.md (monicahq#998)

* Fix sonar issues (monicahq#999)

* Open register page after a clean installation (monicahq#954)

* Redirect to /register if there is no account
* Remove default creation for admin@admin.com

* Fix typo on readme.md (monicahq#1000)

* Update axios to version 0.18.0 (monicahq#914)

* Update axios to version 0.18.0 (monicahq#1006)

See monicahq#914

* Update vue-i18n to version 7.6.0 (monicahq#1007)

* Update laravel-mix to version 2.1.7 (monicahq#1009)

* Reduce sonar complexity (monicahq#992)

* Fix sonar bug (monicahq#1011)

* Reduce precision in codecov setting (monicahq#1012)

* Fix bug with return values (monicahq#1013)

* Remove MoneyHelper from aliases (monicahq#1014)

* Fix notification messages (monicahq#1016)

* Add ability to sort tasks by completed_at attribute (monicahq#1021)

* Add sorting capabilities to most models in the API (monicahq#1022)

* Update Czech, Italian, Portuguese, Russian, German, French, Chinese Simplified, Hebrew language files (monicahq#966)

*  Add two new languages: Hebrew and Chinese Simplified (monicahq#1023)

* Bump version to 1.8.2 (monicahq#1024)

* Replace logo in README (monicahq#1027)

* Fix vagrant install using bash (monicahq#1029)

* Update i18n assets on build (monicahq#1032)

* Bump enm1989/chromedriver from 2.36 to 2.37 (monicahq#1037)

Bumps [enm1989/chromedriver](https://github.com/ENM1989/chromedriver) from 2.36 to 2.37.
- [Release notes](https://github.com/ENM1989/chromedriver/releases)
- [Commits](ENM1989/chromedriver@2.36...2.37)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Bump barryvdh/laravel-debugbar from 3.1.3 to 3.1.4 (monicahq#1034)

Bumps [barryvdh/laravel-debugbar](https://github.com/barryvdh/laravel-debugbar) from 3.1.3 to 3.1.4.
- [Release notes](https://github.com/barryvdh/laravel-debugbar/releases)
- [Changelog](https://github.com/barryvdh/laravel-debugbar/blob/master/changelog.md)
- [Commits](barryvdh/laravel-debugbar@v3.1.3...v3.1.4)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Bump pragmarx/google2fa from 3.0.0 to 3.0.1 (monicahq#1039)

Bumps [pragmarx/google2fa](https://github.com/antonioribeiro/google2fa) from 3.0.0 to 3.0.1.
- [Release notes](https://github.com/antonioribeiro/google2fa/releases)
- [Changelog](https://github.com/antonioribeiro/google2fa/blob/master/changelog.md)
- [Commits](antonioribeiro/google2fa@v3.0.0...v3.0.1)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Add linkedin url in the Contact object returned by the API (monicahq#1042)

* Fix called_at field in the Call object returned by the API (monicahq#1043)

* Add with parameter in Contact object to retrieve contact fields (monicahq#1044)

* Add yearly statistics to activities and calls object (monicahq#1049)

* Bump jeroendesloovere/vcard from 1.5.2 to 1.6 (monicahq#1038)

Bumps [jeroendesloovere/vcard](https://github.com/jeroendesloovere/vcard) from 1.5.2 to 1.6.
- [Release notes](https://github.com/jeroendesloovere/vcard/releases)
- [Changelog](https://github.com/jeroendesloovere/vcard/blob/master/CHANGELOG.md)
- [Commits](jeroendesloovere/vcard@1.5.2...1.6)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Bump symfony/dom-crawler from 3.0.9 to 4.0.6 (monicahq#1033)

* Bump symfony/dom-crawler from 3.0.9 to 4.0.6
* Bump symfony/css-selector from 3.0.9 to 4.0.6

* Symplify return (monicahq#1054)

* Use assertCount (monicahq#1052)

* I18n improvements (monicahq#1060)

* Add a default locale
* Use html lang and dir attributes
* Fix translations
* Add pluralize forms

* Fix i18n improvements (monicahq#1061)

* Fix some codes and pluralize
* Remove urls from locales

* New Crowdin translations (monicahq#1025)

* Add reference to Crowdin in the documentation (monicahq#1066)

* Bump guzzlehttp/guzzle from 6.3.0 to 6.3.2 (monicahq#1063)

Bumps [guzzlehttp/guzzle](https://github.com/guzzle/guzzle) from 6.3.0 to 6.3.2.
- [Release notes](https://github.com/guzzle/guzzle/releases)
- [Changelog](https://github.com/guzzle/guzzle/blob/master/CHANGELOG.md)
- [Commits](guzzle/guzzle@6.3.0...6.3.2)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Use lang list instead of auto (monicahq#1068)

* Use unsigned* columns (monicahq#1053)

* Use now helper (monicahq#1051)

* Fix i18n support (monicahq#1069)

Fix i18n quotation
Add hreflang on links
Fix lang list on settings which does not appear ! Bug introduced with monicahq#1068

* New Crowdin translations (monicahq#1064)

* Bump league/flysystem-aws-s3-v3 from 1.0.18 to 1.0.19 (monicahq#1067)

Bumps [league/flysystem-aws-s3-v3](https://github.com/thephpleague/flysystem-aws-s3-v3) from 1.0.18 to 1.0.19.
- [Release notes](https://github.com/thephpleague/flysystem-aws-s3-v3/releases)
- [Changelog](https://github.com/thephpleague/flysystem-aws-s3-v3/blob/master/changelog.md)
- [Commits](thephpleague/flysystem-aws-s3-v3@1.0.18...1.0.19)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Fixed usage of wrong success message when adding existing contact as kid of someone (monicahq#1041)

* Bump creativeorange/gravatar from 1.0.10 to 1.0.11 (monicahq#1074)

* Bump creativeorange/gravatar from 1.0.10 to 1.0.11

Bumps [creativeorange/gravatar](https://github.com/creativeorange/gravatar) from 1.0.10 to 1.0.11.
- [Release notes](https://github.com/creativeorange/gravatar/releases)
- [Commits](creativeorange/gravatar@v1.0.10...v1.0.11)

Signed-off-by: dependabot[bot] <support@dependabot.com>

* Order lastname and middlename parameters correctly for contact->setName call (monicahq#1073)

* Update i18n messages (monicahq#1076)

* Use phpdbg and other test improvements (monicahq#1082)

* Bump google2fa-laravel to version 0.2.0 (monicahq#1084)

* Fix a duplicate key message (monicahq#1085)

* Split app.js and generate separate javascript language files (monicahq#1079)
@github-actions
Copy link

This pull request has been automatically locked since there
has not been any recent activity after it was closed.
Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant