#196 API call of asignees

Closed
opened 1 month ago by 6543 · 4 comments
6543 commented 1 month ago

"assignee": "string" is deprecated

use "assignees": [ "string" ] instead

-> use at creat issue and more?

`"assignee": "string"` is deprecated use `"assignees": [ "string" ]` instead -> use at creat issue and more?
6543 added this to the 2.4.0 milestone 1 month ago
6543 added the
Refactor
label 1 month ago
6543 closed this issue 1 month ago
6543 self-assigned this 1 month ago
mmarif commented 1 month ago
Owner

@6543 assignee is removed from the APIs?

Which version of Gitea will have this in effect?

If you check the Issues model, it have both assignee and assignees.

https://gitea.com/gitnex/GitNex/src/branch/master/app/src/main/java/org/mian/gitnex/models/Issues.java

At most it will effect creating new issue because the assignee object is required.

@6543 `assignee` is removed from the APIs? Which version of Gitea will have this in effect? If you check the Issues model, it have both assignee and assignees. https://gitea.com/gitnex/GitNex/src/branch/master/app/src/main/java/org/mian/gitnex/models/Issues.java At most it will effect creating new issue because the assignee object is required.
6543 commented 1 month ago
Poster

It wont be removed - it’s not recomend to use it ...

I rewrote the function who parse this ... gitea/#8873

And noticed that assignee is only for backwarts compatibilety ...

So:

It wont be removed - it's not recomend to use it ... I rewrote the function who parse this ... [gitea/#8873](https://github.com/go-gitea/gitea/pull/8873) And noticed that assignee is only for backwarts compatibilety ... So: * [ ] switch to `assignees` only * [ ] asign the current user when creating a issue should be optional (noticed at debugging ...) - current user should be selectable at assignees list - remove default asign ...
mmarif commented 1 month ago
Owner

It need to be implemented that way to not break the current flow for old versions as that PR is merged to 1.11 and assignee is required for old versions(even it should not as you mentioned).

But it is not really a required thing at this moment. If we don’t implement it until 1.11 release, anything will break?

We can make it optional, but somehow we need to know the API accept it as optional not mandatory.

asign the current user when creating a issue should be optional

This is the important bit because Gitea until 1.10 make it mandatory if I am not mistaken.

It need to be implemented that way to not break the current flow for old versions as that PR is merged to 1.11 and assignee is required for old versions(even it should not as you mentioned). But it is not really a required thing at this moment. If we don't implement it until 1.11 release, anything will break? We can make it optional, but somehow we need to know the API accept it as optional not mandatory. > asign the current user when creating a issue should be optional This is the important bit because Gitea until 1.10 make it mandatory if I am not mistaken.
6543 commented 1 month ago
Poster

I’ll test it ... and make a PR

I'll test it ... and make a PR
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.