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

Truncate commit message on branches and prebuilds #6074

Merged
merged 1 commit into from
Oct 14, 2021

Conversation

gtsiolis
Copy link
Contributor

@gtsiolis gtsiolis commented Oct 6, 2021

Description

This will truncate the commit message on branches and prebuilds.

Related Issue(s)

Fix #5335

Screenshots

BEFORE AFTER
Screenshot 2021-10-06 at 6 39 42 PM (2) Screenshot 2021-10-06 at 6 39 55 PM (2)

How to test

  1. Add a project with a long commit message on the default branch
  2. Trigger a prebuild for the default branch
  3. Check out the branches and prebuilds pages for that project

Release Notes

Truncate commit message on branches and prebuilds

@gtsiolis gtsiolis added component: dashboard type: improvement Improves an existing feature or existing code feature: teams and projects [DEPRECATED] Please, use feature: organizations or feature: projects labels instead. team: webapp Issue belongs to the WebApp team labels Oct 6, 2021
@roboquat roboquat requested a review from geropl October 6, 2021 15:17
<div>
<div className="text-base text-gray-500 dark:text-gray-50 font-medium mb-1">{shortCommitMessage(p.info.changeTitle)}</div>
<div className="truncate">
<div className="text-base text-gray-500 dark:text-gray-50 font-medium mb-1 truncate">{shortCommitMessage(p.info.changeTitle)}</div>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

thought: Not sure if I'm missing something here but this works better when using the truncate class on both elements. 🤷

@gtsiolis
Copy link
Contributor Author

gtsiolis commented Oct 11, 2021

@geropl could you take a look at this small change when you're free?

priority: Low 🔅

@geropl
Copy link
Member

geropl commented Oct 14, 2021

@gtsiolis Sorry for this taking so long. I will try to improve my review workflow with the next free slot I can find 🙇

Copy link
Member

@geropl geropl left a comment

Choose a reason for hiding this comment

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

Changes look good!

@roboquat
Copy link
Contributor

LGTM label has been added.

Git tree hash: e48d7356e043c2fb5cba2dd257ba68bd05da363f

@roboquat
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: geropl

Associated issue: #5335

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@roboquat roboquat merged commit 7840e68 into main Oct 14, 2021
@roboquat roboquat deleted the gt/truncate-commit-message branch October 14, 2021 12:49
@roboquat roboquat added deployed: webapp Meta team change is running in production deployed Change is completely running in production labels Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved component: dashboard deployed: webapp Meta team change is running in production deployed Change is completely running in production feature: teams and projects [DEPRECATED] Please, use feature: organizations or feature: projects labels instead. release-note size/XS team: webapp Issue belongs to the WebApp team type: improvement Improves an existing feature or existing code user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Truncate commit message on the branches and prebuilds pages
3 participants