manual-approval icon indicating copy to clipboard operation
manual-approval copied to clipboard

Feature/issue label and file preview

Open reynandaptr opened this issue 2 years ago • 8 comments

  • Allow user set labels of issue
  • Allow user see the file preview before continue the pipeline

Example:

labels: migration
additional-issue-body-file: ZW52aXJvbm1lbnQ6IHRlc3QKc3FsczoKICAgIC0gQUxURVIgVEFCTEUgInN0b3BzIiBBTFRFUiBDT0xVTU4gImNvb3JkaW5hdGUiIFRZUEUgZ2VvbWV0cnkoUG9pbnQsNDMyNikgVVNJTkcgImNvb3JkaW5hdGUiOjpnZW9tZXRyeShQb2ludCw0MzI2KQogICAgLSBBTFRFUiBUQUJMRSAic3RvcHMiIEFMVEVSIENPTFVNTiAiY29vcmRpbmF0ZSIgU0VUIERFRkFVTFQgbnVsbAo=.yaml

image

reynandaptr avatar Apr 10 '23 05:04 reynandaptr

Thanks for your your contribution! I'm a little confused what the use case is for the additional issue body file?

trstringer avatar Apr 23 '23 15:04 trstringer

I find the label feature quite useful for automated slack notifications, maybe get that in separately from the issue body file feature?

rrrkren avatar Jul 05 '23 21:07 rrrkren

I think that's a great idea! @reynandaptr, would you be willing to break this up into two different PRs?

trstringer avatar Oct 19 '23 02:10 trstringer

I think that's a great idea! @reynandaptr, would you be willing to break this up into two different PRs?

I will do it

reynandaptr avatar Nov 05 '23 15:11 reynandaptr

@trstringer could you review my new PR for the labels feature? Thanks

reynandaptr avatar Nov 05 '23 15:11 reynandaptr

@reynandaptr Thanks for contributing! In order to proceed with merging, please:

  • update this PR title to accurately reflect the contents of the change
  • rebase this and the other PR you have pending

lizziemac avatar Feb 24 '25 14:02 lizziemac

Hi @reynandaptr

Could you please update your PR based on the above two points. TIA

snskArora avatar Jun 07 '25 17:06 snskArora

Also please sync you branch and remove the changes for adding labels.

snskArora avatar Jun 13 '25 21:06 snskArora