Skip to content
Snippets Groups Projects

Features/profil page

Merged Guilhem CARRON requested to merge features/profil_page into dev
All threads resolved!

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Yoan VALLETYoan VALLET 4 years ago (Jul 22, 2020 4:10pm UTC)

Merge details

  • Changes merged into dev with 898666e9.
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Guilhem CARRON added 1 commit

    added 1 commit

    • fb46af53 - add translation + change enrgyRecap to Report

    Compare with previous version

  • Guilhem CARRON added 1 commit

    added 1 commit

    Compare with previous version

  • 1/ The report information is well managed in the db but not in the context. If you navigate to another screen and come back to the profil, none option is selected.

    You should create an setUserProfil method in the context in order to update it.

    2/ You don't need to create a state which store the report state as you use the hook to the context. When the value will change in the context, it will update your component.

    Edited by Yoan VALLET
  • Guilhem CARRON added 1 commit

    added 1 commit

    • 036b5f01 - Add userprofile update in context

    Compare with previous version

  • Guilhem CARRON added 1 commit

    added 1 commit

    Compare with previous version

  • Guilhem CARRON added 2 commits

    added 2 commits

    • ffe72009 - Fix desktop version
    • cd3a0eb9 - Merge branch 'features/profil_page' of...

    Compare with previous version

  • Yoan VALLET resolved all threads

    resolved all threads

  • merged

  • Yoan VALLET mentioned in commit 898666e9

    mentioned in commit 898666e9

  • Please register or sign in to reply
    Loading