-
Notifications
You must be signed in to change notification settings - Fork 65
✨ Hide visibility of check-ins for individual users #4246
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
base: develop
Are you sure you want to change the base?
Conversation
|
Ich sehe, dass der Test unter anderem wegen der Datenbank fehlschlägt. Bei meinem lokalen Test hat allerdings alles funktioniert. Wie ist dieses Problem zu lösen? MfG. |
|
Did you push all the files? At least one file is missing, e.g., the migration. |
Possibly I forgot it! I will look for the missing files as soon as I am back home. :) |
|
All files should now be uploaded. @MrKrisKrisu |
|
I see that a check is still failing, which is strange since everything starts normally in my local instance. What can I do? @MrKrisKrisu |
|
I've added the last missing file that I have forgotten, sorry! |
|
I think everything should be okay now. @MrKrisKrisu :) |
|
With the last commits I fixed all "quality standard" issues. @MrKrisKrisu |
Unfortunately, I won't be able to review this PR over the holidays. Perhaps someone else here has time, or you will need to wait until next year. |
All good, I have no problem waiting. I wish you happy christmas days. ^^ |
|
This is explicitly not a review yet, I just skimmed through it, but here are a few comments:
|
|
Thank you for informing me about the UUID/ID issue, I will fix it as soon as possible. Of course I test the PR before submit it, but on my local machine the testing possibilities are slightly limited. I will try to make the UI more minimalistic and to integrate it better in the empty space. |
Added the option to hide a check-in for a user directly at the check-in itself, and not only in the window where you can edit the check-in.
The bug that you were able to select yourself in the hidden user menu.
|
Hey @MrKrisKrisu, I think now the PR is ready for review again. |
|
With the last commit I fixed the missing spaces at line 75 in "DashboardController.php". Now all "quality standards" should have been met. @MrKrisKrisu |



This new feature allows you to prevent individual users from seeing specific check-ins. This can be customized for each check-in individually. Users added to a check-in via this feature's menu will not be able to see that check-in on the user's "En Route," "Dashboard," or "Profile" pages.
see #4242