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

Adding filter-buttons in pod list view #21

Merged
merged 2 commits into from Apr 5, 2024

Conversation

tclaus
Copy link
Owner

@tclaus tclaus commented Apr 5, 2024

Style fixes

"click #show_all_pods": "showAllPods",
"click #show_active_pods": "showActivePods",
"click #show_blocked_pods": "showBlockedPods",
"click #show_invalid_pods": "showInvalidPods",
Copy link

Choose a reason for hiding this comment

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

comma-dangle: Unexpected trailing comma.


initialize: function() {
this.pods = new app.collections.Pods(app.parsePreload("pods"));
this.rows = []; // contains the table row views
this.podfilter = 'active';
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.

},

showAllPods: function() {
this.podfilter = '';
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.

},

showActivePods: function() {
this.podfilter = 'active';
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.

},

showBlockedPods: function() {
this.podfilter = 'blocked';
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.

}).render());
this.$("tbody").empty();

this.pods.each(function (pod) {
Copy link

Choose a reason for hiding this comment

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

space-before-function-paren: Unexpected space before function parentheses.

this.$("tbody").empty();

this.pods.each(function (pod) {
if (self.podfilter === '' ||
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.


this.pods.each(function (pod) {
if (self.podfilter === '' ||
self.podfilter === 'active' && pod.get("status") === "no_errors" ||
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.

this.pods.each(function (pod) {
if (self.podfilter === '' ||
self.podfilter === 'active' && pod.get("status") === "no_errors" ||
self.podfilter === 'blocked' && pod.get("blocked") === true ||
Copy link

Choose a reason for hiding this comment

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

quotes: Strings must use doublequote.

if (self.podfilter === '' ||
self.podfilter === 'active' && pod.get("status") === "no_errors" ||
self.podfilter === 'blocked' && pod.get("blocked") === true ||
self.podfilter === 'invalid' && pod.get("status") !== "no_errors") {
Copy link

Choose a reason for hiding this comment

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

  • quotes: Strings must use doublequote.
  • padded-blocks: Block must not be padded by blank lines.

@tclaus tclaus force-pushed the feature/filter_pods_in_admnin_view branch from b774019 to 417187c Compare April 5, 2024 14:33
@tclaus tclaus merged commit fef0a72 into development Apr 5, 2024
5 of 13 checks passed
@tclaus tclaus deleted the feature/filter_pods_in_admnin_view branch April 5, 2024 14:54
tclaus added a commit that referenced this pull request Apr 6, 2024
* Filter buttons for Pod view
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant