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

feat(ui) Reworked the Trigger page to show only columns that have data #5555

Merged
merged 10 commits into from
Oct 24, 2024

Conversation

MRADUL9936
Copy link
Contributor

What changes are being made and why?

made changes in ui/src/components/admin/triggers.vue , because this is the parent component which is responsible for
fetching the data from store and rendering the table .
I am using a method to filter out the columns which is having no data , and rendering these columns.

Screenshot (253)

How the changes have been QAed?

Tested in Different Browsers

closes #5546

@MilosPaunovic MilosPaunovic self-requested a review October 20, 2024 18:22
@MilosPaunovic MilosPaunovic self-assigned this Oct 20, 2024
Copy link
Member

@MilosPaunovic MilosPaunovic left a comment

Choose a reason for hiding this comment

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

Le't try to keep the formatting changes to a minimum, as it will be easier to review PR.

ui/src/components/admin/Triggers.vue Show resolved Hide resolved
</template>
</el-table-column>

<el-table-column column-key="disable" class-name="row-action">
Copy link
Member

Choose a reason for hiding this comment

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

Also, this column seem missing too.

@Skraye
Copy link
Member

Skraye commented Oct 21, 2024

@MRADUL9936 Note that Anna do not ask to remove the column, only to hide them if they are empty, however they are still very important

@MilosPaunovic MilosPaunovic self-requested a review October 23, 2024 22:17
@MRADUL9936
Copy link
Contributor Author

MRADUL9936 commented Oct 23, 2024

@MilosPaunovic I made changes , can u please review it now . , OUTPUT :
Screenshot (270)

Copy link
Member

@MilosPaunovic MilosPaunovic left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

@MilosPaunovic MilosPaunovic merged commit d33d6a7 into kestra-io:develop Oct 24, 2024
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Rework the Trigger page to show only columns that have data
3 participants