Skip to content
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

[MNOE-474] Update review frontend #219

Merged
merged 5 commits into from
Jun 26, 2017

Conversation

adamaziz15
Copy link
Contributor

@adamaziz15 adamaziz15 commented Jun 15, 2017

@ouranos ouranos requested a review from alexnoox June 16, 2017 07:52
</div>
<div class="pull-left links">
<a ng-show="vm.anyReviews" ng-click="vm.scrollToReviews()" href="">{{vm.reviews.list.length}} {{'mno_enterprise.templates.dashboard.marketplace.show.reviews' | translate}}</a>
<span ng-if="vm.anyReviews && vm.anyQuestions"> | </span>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use &nbsp;: &nbsp;|&nbsp;
I'm afraid that the code uglifier will remove those spaces after a build.

@adamaziz15
Copy link
Contributor Author

@alexnoox replaced spaces with &nbsp;

@alexnoox alexnoox closed this Jun 20, 2017
@alexnoox alexnoox reopened this Jun 20, 2017
@alexnoox
Copy link
Contributor

@ouranos Please port locales to mno-enterprise and merge

Copy link
Contributor

@alexnoox alexnoox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please create a PR in mno-enterprise for the locales

@@ -519,6 +521,9 @@
"mno_enterprise.templates.dashboard.marketplace.show.comment.delete_modal_body": "Do you really want to delete your comment?",
"mno_enterprise.templates.dashboard.marketplace.show.comment.delete_modal_cancel": "Cancel",
"mno_enterprise.templates.dashboard.marketplace.show.comment.delete_modal_delete": "Delete",
"mno_enterprise.templates.dashboard.marketplace.show.comment.hide": "Hide Comments",
"mno_enterprise.templates.dashboard.marketplace.show.comment": "Comment",
"mno_enterprise.templates.dashboard.marketplace.show.comments": "Comments",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can use pluralisation to manage this (https://angular-translate.github.io/docs/#/guide/14_pluralization)

@@ -519,6 +521,9 @@
"mno_enterprise.templates.dashboard.marketplace.show.comment.delete_modal_body": "Do you really want to delete your comment?",
"mno_enterprise.templates.dashboard.marketplace.show.comment.delete_modal_cancel": "Cancel",
"mno_enterprise.templates.dashboard.marketplace.show.comment.delete_modal_delete": "Delete",
"mno_enterprise.templates.dashboard.marketplace.show.comment.hide": "Hide Comments",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rename this key to mno_enterprise.templates.dashboard.marketplace.show.hide_comments. If a key is used (ie. mno_enterprise.templates.dashboard.marketplace.show.comment) we cannot reuse it by adding .hide. It has to be translated to YAML and this format do not accept it.

@ouranos ouranos changed the base branch from master to 1.1 June 22, 2017 07:15
@adamaziz15 adamaziz15 force-pushed the feature/474-refresh-review-frontend branch from be59cab to ae9c957 Compare June 22, 2017 19:44
@adamaziz15
Copy link
Contributor Author

@alexnoox updated locales and view to use pluralization

@adamaziz15
Copy link
Contributor Author

@alexnoox Updated locales and created PR in mnoe. Please review.

@ouranos
Copy link
Contributor

ouranos commented Jun 25, 2017

@alexnoox locales all good for me. Let's merge in 1.1 and I'll include it before my big 2.0 refactor

@alexnoox
Copy link
Contributor

@ouranos Ok, let's do it

@ouranos
Copy link
Contributor

ouranos commented Jun 26, 2017

Locales merged in maestrano/mno-enterprise#334

@ouranos ouranos merged commit d797c1f into maestrano:1.1 Jun 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants