Fix hide calender icon if no deadline on task

[Vardhaman619]

Closes #7402

Ensured that the calendar icon is only visible when a deadline is assigned

[greptile-apps]

PR Summary

This pull request addresses the issue of displaying a calendar icon for tasks without deadlines. The main changes are focused on improving the task display in the Twenty front-end application.

• Modified TaskRow.tsx to conditionally render the calendar icon and due date only when task.dueAt is present
• Adjusted layout in SettingsApiKeysFieldItemTableRow.tsx, SettingsApiKeysTable.tsx, and SettingsDevelopersWebhookTableRow.tsx for improved responsiveness
• Updated grid template columns in settings components to use more flexible layouts
• Removed unnecessary React import in SettingsDevelopersWebhookTableRow.tsx

These changes enhance the user interface by removing visual clutter and improving the overall layout of various components in the application.

_{4 file(s) reviewed, 2 comment(s)
Edit PR Review Bot Settings}

[greptile-apps]

style: Redundant check for task.dueAt, as it's already verified in the conditional rendering

[Bonapara]

@Vardhaman619, could you move the due date to the left of the relation field on the card?

[Vardhaman619]

@Vardhaman619, could you move the due date to the left of the relation field on the card?

@Bonapara Done 👍

[ehconitin]

Could u fix the conflicts @Vardhaman619? or do you need help?

[Vardhaman619]

@ehconitin I really want to fix this conflict, but I'm not sure where to start. Could you please outline the steps and processes I should follow to resolve it? Your guidance would be really helpful.

[ehconitin]

@Vardhaman619 We merged your previous PR #7438 yesterday, but for some reason same changes were included in this PR as well. The merged changes differ slightly from what you’ve added here, so I’ve removed those redundant changes to resolve the conflicts.
:)
This was a straightforward case, but conflicts like this can become more complex. To avoid them, I recommend keeping your branch up to date with the upstream branch as frequently as possible. ;)

[Vardhaman619]

@ehconitin Thank you for handling the conflicts and for the helpful advice! I'll make sure to keep my branch up to date with the upstream branch more frequently to avoid such issues in the future.

Thanks

[ehconitin]

@Bonapara instead of showing due date after task target, should we show it before?
That way task target would be consistently shown last, let me know what do you think!
due date after task target -

due date before task target -

[Bonapara]

@ehconitin yes we should show it before!

[oss-gg]

Awarding Vardhaman619: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/Vardhaman619

[github-actions]

Thanks @Vardhaman619 for your contribution!
This marks your 5th PR on the repo. You're top 5% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter