Add server side permalink validation for users #66
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
I found missing values on db object Teaser events (avnode/app/server/views/user/teaser/events.jade) date time venue city country Teaser crew (app/server/views/user/teaser/crews.jade) mancano un po’ di info sulle crew nell’oggetto performer.crews performances number events number
I found missing values on db object Teaser events (avnode/app/server/views/user/teaser/events.jade) date time venue city country Teaser crew (app/server/views/user/teaser/crews.jade) mancano un po’ di info sulle crew nell’oggetto performer.crews performances number events number
Missing data on footage item app/server/views/footage/item.jade are: size value tags/category thumbnail 16:9 cropped thumbnail Missing data on playlist item app/server/views/playlists/item.jade are: Likes value Videos inside the playlist Category/tag of the video thumbnail 16:9 cropped thumbnail Missing data on tvshow item app/server/views/tvshows/item.jade are: likes value Category/tag of the video thumbnail 16:9 cropped thumbnail Missing data on gallery app/server/views/gallery/item.jade are: video number (I can find it but i’m not able to show it with box.stats.video) gallery typology (es “performance”) location city country 16:9 cropped thumbnail
1 - Developing design in beahnce style
2 - new way for calling items in order to decide how many columns must have the list:
create a variable in app/server/views/user/show.jade (and similars)
each box, i in performer.footage
if (i > 0 && i < 10)
- var col_number = "col-xs-6"
include ../footage/item
move the content of the item inside a mixin app/server/views/footage/item.jade (and similars)
mixin footage_item_content
a(href="/#{box.users[0].permalink}/footage/#{box.permalink}/", title=box.title)
.image_box.wrapper_item
each file, i in box.files (ETC ETC….)
Set the if statement depending on variable presence
if (col_number)
div.list_item_performer.grid-item.footage_footage(class= col_number)
+footage_item_content
else
div.list_item_performer.grid-item.col-sm-4(class="col-md-4 footage_footage")
+footage_item_content
Set the variable in lists template app/server/views/user/sections/footage.jade (and similars)
extends ../layout
block user-content
// template: app/server/views/user/sections/footage.jade
.user_lists_bkg
.container
h1.user_list_title=__("Events")
.row.grid.user_list
each box in performer.footage
- var col_number = "col-sm-3"
include ../../footage/item
- use jquery.equalheights.js in user template - restyle of single event page app/server/views/user/events/show.jade
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.
Ensures a permalink is unique. Related to #45