ge_issue_comment looks a bit cluttered with the if statements. I'm not sure how to refactor and make it look cleaner. Any help is appreciated. Thanks.
added textControllerto ge_issue_comment. Easier to deal with updation of a comment
(Just saw that Github markdown supports video as well. Wanted to see how it looks. Hence the video below (also easier instead of copy-pasting multiple screenshots))
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.
## 📋 Pull Request Information
**Original PR:** https://github.com/pd4d10/git-touch/pull/150
**Author:** [@shreyas1599](https://github.com/shreyas1599)
**Created:** 1/8/2021
**Status:** ✅ Merged
**Merged:** 1/9/2021
**Merged by:** [@pd4d10](https://github.com/pd4d10)
**Base:** `main` ← **Head:** `gitee-issue-actions`
---
### 📝 Commits (2)
- [`4dfd2b9`](https://github.com/pd4d10/git-touch/commit/4dfd2b9d05c6528d20b5f67c2d7e3072760cb455) feat: comment actions (issue/PR)
- [`911fa27`](https://github.com/pd4d10/git-touch/commit/911fa27174a37c8e24a1f135bdb63aa152a70303) icons for url actions
### 📊 Changes
**11 files changed** (+234 additions, -70 deletions)
<details>
<summary>View changed files</summary>
📝 `lib/models/auth.dart` (+43 -15)
📝 `lib/models/gitee.dart` (+1 -0)
📝 `lib/models/gitee.g.dart` (+2 -0)
📝 `lib/router.dart` (+16 -6)
📝 `lib/screens/ge_commit.dart` (+0 -1)
📝 `lib/screens/ge_issue.dart` (+44 -8)
📝 `lib/screens/ge_issue_comment.dart` (+49 -23)
📝 `lib/screens/ge_issue_form.dart` (+1 -1)
📝 `lib/screens/ge_pull.dart` (+44 -8)
📝 `lib/widgets/action_button.dart` (+21 -7)
📝 `lib/widgets/comment_item.dart` (+13 -1)
</details>
### 📄 Description
- `Edit` and `Delete` actions in comments of `Issues` and `Pull Requests`
- Update `ActionButton` to use icons
- `CommentItem` has a prop `commentActionItemList`. (TODO: for services except gitee)
- minor fix: removed a `print` statement from `GeCommit`
- added `PATCH` and `DELETE` HTTP request types. Refactored `fetchGitee`and added prop `requestType`
- _**`ge_issue_comment` looks a bit cluttered with the `if` statements. I'm not sure how to refactor and make it look cleaner. Any help is appreciated. Thanks.**_
- added `textController`to `ge_issue_comment`. Easier to deal with updation of a comment
_(Just saw that Github markdown supports video as well. Wanted to see how it looks. Hence the video below (also easier instead of copy-pasting multiple screenshots))_
https://user-images.githubusercontent.com/28999492/104002023-7ef08b00-51c6-11eb-9378-b02e990b3b54.mp4
---
<sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
📋 Pull Request Information
Original PR: https://github.com/pd4d10/git-touch/pull/150
Author: @shreyas1599
Created: 1/8/2021
Status: ✅ Merged
Merged: 1/9/2021
Merged by: @pd4d10
Base:
main← Head:gitee-issue-actions📝 Commits (2)
4dfd2b9feat: comment actions (issue/PR)911fa27icons for url actions📊 Changes
11 files changed (+234 additions, -70 deletions)
View changed files
📝
lib/models/auth.dart(+43 -15)📝
lib/models/gitee.dart(+1 -0)📝
lib/models/gitee.g.dart(+2 -0)📝
lib/router.dart(+16 -6)📝
lib/screens/ge_commit.dart(+0 -1)📝
lib/screens/ge_issue.dart(+44 -8)📝
lib/screens/ge_issue_comment.dart(+49 -23)📝
lib/screens/ge_issue_form.dart(+1 -1)📝
lib/screens/ge_pull.dart(+44 -8)📝
lib/widgets/action_button.dart(+21 -7)📝
lib/widgets/comment_item.dart(+13 -1)📄 Description
EditandDeleteactions in comments ofIssuesandPull RequestsActionButtonto use iconsCommentItemhas a propcommentActionItemList. (TODO: for services except gitee)printstatement fromGeCommitPATCHandDELETEHTTP request types. RefactoredfetchGiteeand added proprequestTypege_issue_commentlooks a bit cluttered with theifstatements. I'm not sure how to refactor and make it look cleaner. Any help is appreciated. Thanks.textControllertoge_issue_comment. Easier to deal with updation of a comment(Just saw that Github markdown supports video as well. Wanted to see how it looks. Hence the video below (also easier instead of copy-pasting multiple screenshots))
https://user-images.githubusercontent.com/28999492/104002023-7ef08b00-51c6-11eb-9378-b02e990b3b54.mp4
🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.