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

Add emoji in label to project boards #13978

Merged
merged 3 commits into from
Dec 14, 2020
Merged

Conversation

rakshith-ravi
Copy link
Contributor

As title.

Fixes #13880

CC @modmew8

Added rendering of emoji to project label
@codecov-io
Copy link

Codecov Report

Merging #13978 (a6923fb) into master (633dc80) will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #13978   +/-   ##
=======================================
  Coverage   42.21%   42.22%           
=======================================
  Files         710      710           
  Lines       77233    77233           
=======================================
+ Hits        32603    32609    +6     
- Misses      39252    39254    +2     
+ Partials     5378     5370    -8     
Impacted Files Coverage Δ
models/repo_mirror.go 2.38% <0.00%> (-11.91%) ⬇️
modules/cron/tasks_basic.go 87.35% <0.00%> (-3.45%) ⬇️
models/unit.go 46.57% <0.00%> (-2.74%) ⬇️
modules/process/manager.go 72.50% <0.00%> (-2.50%) ⬇️
modules/log/event.go 59.90% <0.00%> (-1.89%) ⬇️
services/mirror/mirror.go 15.78% <0.00%> (-1.11%) ⬇️
models/issue_comment.go 52.71% <0.00%> (+0.45%) ⬆️
models/error.go 39.31% <0.00%> (+0.81%) ⬆️
models/notification.go 67.24% <0.00%> (+0.98%) ⬆️
services/pull/check.go 51.09% <0.00%> (+2.18%) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 633dc80...a6923fb. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Dec 14, 2020
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Dec 14, 2020
@rakshith-ravi
Copy link
Contributor Author

@zeripath should we add this to 1.13.1?

@zeripath
Copy link
Contributor

Yup - I was just checking what the code base looked like on 1.13 - it appears that this can be backported as it stands.

@zeripath zeripath added backport/v1.13 topic/ui Change the appearance of the Gitea UI topic/projects labels Dec 14, 2020
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Dec 14, 2020
@6543 6543 merged commit 729f0f5 into go-gitea:master Dec 14, 2020
@6543
Copy link
Member

6543 commented Dec 14, 2020

@rakshith-ravi pleace send backport 👍

@rakshith-ravi
Copy link
Contributor Author

@6543 how do I do that?

@6543
Copy link
Member

6543 commented Dec 14, 2020

checkout to the release branch & cherry pick 729f0f5

should I do it for you?

@rakshith-ravi
Copy link
Contributor Author

Nah I'll do it. I wanna learn to backport :D

@6543
Copy link
Member

6543 commented Dec 14, 2020

@rakshith-ravi some hints:

git fetch --all
git checkout -f upstream/release/v1.13 -b backport_13978
git cherry-pick 729f0f5f7b
git push -u fork backport_13978

-> create pull

fork is remote to your fork
upstream is remote to official repo

@techknowlogick techknowlogick added this to the 1.14.0 milestone Dec 14, 2020
6543 pushed a commit to 6543-forks/gitea that referenced this pull request Dec 16, 2020
* Update view.tmpl

Added rendering of emoji to project label

* Add RenderEmojiPlain to the title and remove has-emoji

Co-authored-by: zeripath <art27@cantab.net>
@6543
Copy link
Member

6543 commented Dec 16, 2020

@rakshith-ravi did a backport for you -> #14021

@6543 6543 added the backport/done All backports for this PR have been created label Dec 16, 2020
techknowlogick pushed a commit that referenced this pull request Dec 16, 2020
* Update view.tmpl

Added rendering of emoji to project label

* Add RenderEmojiPlain to the title and remove has-emoji

Co-authored-by: zeripath <art27@cantab.net>

Co-authored-by: Rakshith Ravi <rakshith.ravi@gmx.com>
Co-authored-by: zeripath <art27@cantab.net>
@rakshith-ravi
Copy link
Contributor Author

Hey. Thanks. My laptop just broke and I wouldn't have been able to do it till the weekend. Sorry about the delay

@go-gitea go-gitea locked and limited conversation to collaborators Jan 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport/done All backports for this PR have been created lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. topic/projects topic/ui Change the appearance of the Gitea UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Project boards: Emojis in Labels missing
6 participants