#1327

I think you also need to add the default value, eg master in these three files.

@Jocs i considered this question.
with api of octorest lib if u leave branch empty, it will commit for default branch which can be one not master branch...
so the default value is not proper...
how do u think about it？

BYW do you know why did the CI fail this time？
i have no.idea about that....:sob:

it will commit for default branch which can be one not master branch...

Yes, if the branch is empty, just delete it as you do, we can give the default value is '', empty string.

BYW do you know why did the CI fail this time？

I think it's due to something wrong with CI when merge PR from other repo other marktext, it's not due to this PR, so we can merge it anyway after review.

@Jocs yeah, i have tried it before.
when we use empty string, the rest put failed.
in other words, if you define the branch in payload, u cant define it as empty string, or it will fail.

@Jocs i dont get what u mean🤔
i have given branch one default value as '' in view layer in src/ renderer/ prefComponents/ imageUploader/ index. vue🤔.
do u think which is not one proper choice to implement it？ do i need to modify the initialisation in another place？

and thank you very much for your review and explanation.:stuck_out_tongue_closed_eyes:

@Jocs Oh...sorry i missed the pic you pasted before, sorry for that.. i will add it. sorry for that again....

@Jocs Sorry.. I received the issue messages with one android app these days, and that app didn't display the pic you pasted ahead... so i didn't get what u mean before.
i have modify the commit and commit message. thx for your review again..

@Yansongsongsong Thanks for your PR, it looks great.