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

Request description as part of the conversation history #14607

Merged
merged 1 commit into from
Jul 6, 2023

Conversation

ncounter
Copy link
Contributor

@ncounter ncounter commented Jul 5, 2023

In the scope of highlighting more the BsRequest description, it is a fact that the description itself is the initial comment of the request, so it is the root from where the history starts.

With this PR the description has its own title and box, and the history timeline starts from this box.
By doing this, the aside column with all the other information (reviewers, maintainers, etc) are aligned with the top of the Conversation tab panel.

Before

image

After

image

@github-actions github-actions bot added the Frontend Things related to the OBS RoR app label Jul 5, 2023
@ncounter ncounter force-pushed the request-description-text branch 3 times, most recently from 375b1da to 2529d90 Compare July 5, 2023 13:54
@ncounter ncounter marked this pull request as ready for review July 5, 2023 14:05
@codecov
Copy link

codecov bot commented Jul 5, 2023

Codecov Report

Merging #14607 (53bc893) into master (55e89ff) will increase coverage by 2.77%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14607      +/-   ##
==========================================
+ Coverage   84.42%   87.19%   +2.77%     
==========================================
  Files         744      744              
  Lines       25015    25015              
==========================================
+ Hits        21119    21812     +693     
+ Misses       3896     3203     -693     

@ncounter ncounter added the review-app Apply this label if you want a review app started label Jul 5, 2023
@obs-bot
Copy link
Collaborator

obs-bot commented Jul 5, 2023

Review app will appear here: http://obs-reviewlab.opensuse.org/ncounter-request-description-text

@ncounter ncounter merged commit fa4608c into openSUSE:master Jul 6, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend Things related to the OBS RoR app review-app Apply this label if you want a review app started
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants