Changeset - ea284e51ec2e
[Not reviewed]
default
0 2 0
Thomas De Schampheleire - 10 years ago 2015-06-19 15:40:27
thomas.de_schampheleire@alcatel-lucent.com
pullrequest: warn for duplication of reviewer handling with base.js
2 files changed with 6 insertions and 0 deletions:
0 comments (0 inline, 0 general)
kallithea/public/js/base.js
Show inline comments
 
@@ -1473,6 +1473,9 @@ var addReviewMember = function(id,fname,
 
    var gravatarelm = '<img alt="gravatar" style="width: {0}px; height: {0}px" src="{1}"/>'.format(gravatar_size, gravatar_link);
 
    if (!gravatar_link)
 
        gravatarelm = '<i class="icon-user" style="font-size: {0}px;"></i>'.format(gravatar_size);
 
    // WARNING: the HTML below is duplicate with
 
    // kallithea/templates/pullrequests/pullrequest_show.html
 
    // If you change something here it should be reflected in the template too.
 
    var element = (
 
        '     <li id="reviewer_{2}">\n'+
 
        '       <div class="reviewers_member">\n'+
kallithea/templates/pullrequests/pullrequest_show.html
Show inline comments
 
@@ -212,6 +212,9 @@ ${self.repo_context_bar('showpullrequest
 
          <div>
 
            <ul id="review_members" class="group_members">
 
            %for member,status in c.pull_request_reviewers:
 
              ## WARNING: the HTML below is duplicate with
 
              ## kallithea/public/js/base.js
 
              ## If you change something here it should be reflected in the template too.
 
              <li id="reviewer_${member.user_id}">
 
                <div class="reviewers_member">
 
                    <div class="reviewer_status tooltip" title="${h.tooltip(h.changeset_status_lbl(status.status if status else 'not_reviewed'))}">
0 comments (0 inline, 0 general)