-
Notifications
You must be signed in to change notification settings - Fork 80
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
SHARE-555 child restriction #4783
Conversation
f7b07c9
to
e59ffd4
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #4783 +/- ##
=============================================
- Coverage 47.36% 41.15% -6.22%
- Complexity 5926 5940 +14
=============================================
Files 669 670 +1
Lines 20982 21057 +75
=============================================
- Hits 9939 8666 -1273
- Misses 11043 12391 +1348
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
8799d61
to
72b5a50
Compare
CAPI Model/ProjectResponse fehlt noch die Variable not_for_kids, hab sie zur catroweb.yaml file hinzugefügt wurde auch schon gemerged Edit: -> #4799 |
translations/catroweb.en.yaml
Outdated
@@ -1038,5 +1042,6 @@ maintenanceinformations: | |||
feature_2: 'The Feature' | |||
feature_3: 'The Feature' | |||
|
|||
|
|||
MarkNotForKids: Mark as not for kids |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MarkNotForKids: Mark as not for kids | |
MarkNotForKids: Mark as not safe for kids |
translations/catroweb.en.yaml
Outdated
@@ -1027,6 +1028,9 @@ media_library: | |||
|
|||
snackbar: | |||
project_not_found: 'The requested project is not available' | |||
project_not_for_kids: 'The project was marked as not for kids!' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
project_not_for_kids: 'The project was marked as not for kids!' | |
project_not_for_kids: 'The project was marked as safe not for kids!' |
if (null === $project) { | ||
return $this->redirectToIndexOnError(); | ||
} | ||
if (2 == $project->getNotForKids()) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if (2 == $project->getNotForKids()) { | |
if (<const>== $project->getNotForKids()) { |
Use constants please
@makne17 api is now merged, just rebase :) |
added not for kids property to admin panel added button to change not for kids flag added not for kids flag in meta data
added not for kids property to admin panel
added button to change not for kids flag
added not for kids flag in meta data
Your checklist for this pull request
Please review the contributing guidelines and wiki pages of this repository.
SHARE-666 The devils ticket
Code Review
section in JiraAdditional Description
TODO: Add additional information that is not in your commit-message here
Tests - additional information
TODO: add additional information about testruns here