#456 - user profile #464
Merged
#456 - user profile #464
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There was a problem hiding this comment.
- In my opinion, button "Zobacz wszystkie" on the list with vacation request/overtime request should redirect user with right role to vacation requests list/overtime request list filtered by this user.
- I think I will center the text to the avatar
- When user has a lot of devices, the list is really long and it doesn't look nice. Maybe we should show like five devices and add button "Zobacz wszystkie" and this button will redirect user to list of devices filtered by user
# Conflicts: # resources/js/Pages/AssignedBenefits/AssignedBenefits.vue
I would stick with this behavior. |
EwelinaSkrzypacz
approved these changes
Jul 15, 2024
krzysztofrewak
approved these changes
Jul 15, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Should close #456.
Added user profile page with summary almost all data connected with him in app - year scope. Added links where we have user in indexes or places in app which we has thing connected with users.
Preview:
