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

Support for Emoji Icons in Status names #2

Open
thekaushikls opened this issue Apr 24, 2023 · 3 comments
Open

Support for Emoji Icons in Status names #2

thekaushikls opened this issue Apr 24, 2023 · 3 comments

Comments

@thekaushikls
Copy link

Hi, thank you for your GitHubAction, it has been super helpful. I have been playing around with it for sometime now.

I noticed that in one of my projects, I have the Status name contain emoji, and that seems to not be supported.

In the below image, I am updating some of the statuses. But, only the first one (🧱 Planning) seems to be updated.
Depending on the order, only the first status gets updated.

Is there a specific way to include names with emoji?

MoveIssuesToNextIteration-Issue

Can I do something like this?

statuses:
- 🧱 Planning
- 📋 Backlog
@blombard
Copy link
Owner

blombard commented Apr 24, 2023

Hello @thekaushikls 👋

The problem here is how I parse the statuses.

const statuses = core.getInput('statuses').split(',');

So in your case the action is looking for statuses who start with an empty space (except the first one)

The quick fix for you is just to remove the space after the comma.

statuses: 🧱 Planning,📋 Backlog...

But you are right that

statuses:
  - 🧱 Planning
  - 📋 Backlog

seems neater.

So if I find time, I'll update the code to support this structure.

Edit: I updated the README: 1eedfb0

@thekaushikls
Copy link
Author

Hi, thanks for suggesting to remove the white-spaces. I removed the emojis as a temp fix, will try your method.

Also, I dont think it is common practice to name something that starts with a whitespace. I think just trim() should do the trick.

@agarzola
Copy link
Contributor

agarzola commented Jan 2, 2024

While working on a PR for #5, I experimented with the idea of making statuses an array and discovered that action inputs cannot be arrays. In other words, as far as I can tell this doesn’t work:

statuses:
  - Planning
  - Backlog

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants