search

dorbel-tech / dorbel-app

Main dorbel node.js app
https://app.dorbel.com
0 stars 1 forks source link

Adding age and location to tenant profile #1627

Closed virtser closed 7 years ago

virtser commented 7 years ago

closes #1626 closes #1625 closes #1493

Not including adding it into tenant profile view. Still need to enable those permissions in auth0 for Facebook connect.

codecov[bot] commented 7 years ago

Codecov Report

Merging #1627 into develop will increase coverage by 0.02%. The diff coverage is 7.69%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #1627      +/-   ##
===========================================
+ Coverage    31.23%   31.26%   +0.02%     
===========================================
  Files          168      168              
  Lines         3864     3870       +6     
  Branches       810      815       +5     
===========================================
+ Hits          1207     1210       +3     
- Misses        2621     2625       +4     
+ Partials        36       35       -1
Impacted Files Coverage Δ
...onents/Tenants/TenantProfile/TenantProfileEdit.jsx 2.94% <ø> (ø) :arrow_up:
apartments-api/src/services/userProfileService.js 0% <0%> (ø) :arrow_up:
...components/Tenants/TenantProfile/TenantProfile.jsx 5.88% <0%> (-0.25%) :arrow_down:
front-gateway/src/providers/auth/auth0helper.js 13.04% <25%> (+1.13%) :arrow_up:
apartments-api/src/providers/geoProvider.js 88% <0%> (+8%) :arrow_up:

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 8fe0bad...36678fc. Read the comment docs.

amoslande commented 7 years ago

The padding top of the tenant-profile-occupation-field class looks off... maybe apply it only to the first-child? See TenantProfile.scss line 88