-
Notifications
You must be signed in to change notification settings - Fork 84
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
update infra code from mattermost-simple-plungin #28
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
manland
force-pushed
the
27-updateFromMSP
branch
3 times, most recently
from
May 6, 2019 13:00
f8d3856
to
000fb18
Compare
hanzei
reviewed
May 6, 2019
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Glad you are migrating to go modules 👍
Leaved a few comments
raghavaggarwal2308
added a commit
that referenced
this pull request
Jul 18, 2024
…306) * [MI-1847] Created new branch for create-issue-modal * [MI-1847] fixed error while running make check-style * [MI-1847]Review fixes 1. Converted files to .tsx 2. Defined proper types * [MI-1847]Fixed initial srtate of milestone in create_issue.tsx * [MI-1860]Added attach comment to gitlab issue button * [MI-1860]Added "Attach to gilab issue" modal 1. Button in context menu to open "Attach to gitlab issue" modal. 2. Created modal to attach comment to issue. * [MI-1860]Attach comment endpoint 1. Created api endpoint to attach comment to an issue. * [MI-1860]Added post message 1. Added message when comment is created. * [MI-1860]Handled error 1. Handled error while making call for searching issues. * Code formatting 1. Converted file to .tsx. 2. Removed redundant code. * Merge branch 'MI-1847' of github.com:Brightscout/mattermost-plugin-gitlab into MI-1860 * [MI-1860]Pull from MI-1847 1. Fixed error messages. * removed //eslint-diable... * [MI-1860]Review fixes 1. Made first letter of error lowercase 2. Used shorthand 3. fixed lint errors * Merge branch 'MI-1847' of github.com:Brightscout/mattermost-plugin-gitlab into MI-1860 Review fixes: 1. Defined types and converted files to .tsx * [MI-1860]Review Fixes 1. Defined proper types 2. Changed file extension of types file to .d.ts * [MI-1860]Fixes 1. Corrected error messages in api.go. 2. Added EOF. * [MI-1847] Review fixes 1. Moved type "Actions" from index.ts to main file. 2. Converted form_button to fuctional component. 3. Converted gitlab.jsx to .tsx. * [MI-1847]Review Fixes 1. Converted gitlab.tsx to funtional component. 2. Optional chaining while checking for error in loadoptions function. * Fixed CI error 1. Added "react-dom" dependency in package.json * [MI-1847]Fixes 1. Converted class components to functional components. 2. Used absolute paths. * [MI-1847]Review FIxes 1. Removed mapStateToProps/mapDispatchToProps from functional components and used useSelector/useDispatch. * [MI-1860]Review fixes 1. Converted functional componets to class components. 2. used useCallback and useMemo hook. 3. Returned proper status codes. 4. Used absolute paths for imports. * [MI-1847]Review fixes 1. Used BotID for creating post. 2. Used useCallback hooks. 3. Fixed linting errors. * [MI-1847]Converted remaining class components to functional components * [MI-1860]Fixed variable names * [MI-1847]Review fixes 1. Creates a single type for state - "GlobalState". 2. Converted proptypes from interface to type. * [MI-1847]Review fixes 1. Created a hook for getting assignees, milestones and labels. * [MI-1847]Review fixes 1. Removed any type. * [MI-1860]Review fixes 1.Used use callback hook. * [MI-1860]Review fixes 1. Used useMemo and useCallback. 2. Destructured props. * [MI-1860]Review Fixes 1. Changed name of a variable. * [MI-2221] Review fixes (#22) * [MI-2245] Fix failing "Code scanning / CodeQL" in gitlab create issue modal PR (#23) [MI-2245] Fix failing "Code scanning / CodeQL" in GitLab create issue modal PR * Fix lint errors * [MI-2701]: Done the review fixes of a PR: Adding create issue modal #306 (#28) * [MI-2701]: Done the review fixes of a PR: Adding create issue modal #306 * [MI-2701]: Review fixes done 1. Used actual gitlab icon * [MI-2701]: Review fixes done 1. Improved the gitlab icon * [MI-2701]: Review fixes done 1. Updated the viewbox * [MI-2947]: Did the review fixes of a gitlab PR #306 (#31) * [MI-2967]: Did the review fixes of gitlab PR #306 (#33) * [MI-2967]: Did the review fixes of gitlab PR #306 * [MI-2967]: Improved code readability * [MI-2967]: Review fixes done 1. Changed the name of a variable * [MI-2959] Review fixes on Gitlab PR #306(Add create issue modal) 1. Fixed theme issue in post action menu. 2. Fixed the max length of issue title. 3. Added check to display the projects of connected gitlab group only. 4. Added check to display the issues of connected gitlab group only. * [MI-3787] Fix reported issue: - Hide the additional fields in create issue modal if the selected priject is removed. * [MM-3] Removed "ussCallback" and "useMemo" hooks * [MM-3] Removed useMemo/useCallabck and fixed lint error. * [MM-3] Separated out form components for create issue modal * [MM-3] Seperate out form component for attach comment to issue modal * Fix lint errors * [MM-158] Review fixes 1. Rename types 2. Create selectors for modal visibility vars 3. Fix warnings in console * [MM-158] Review fixes: 1. Mode the footer part to form components 2. Create selectors for getting modal contents 3. Make the meesage field in attach comment modal as editable and added validation * [MM-158] Review fixes: 1. Create a common file for types. 2. Moved the modal form components to modals directory. * [MM-146] Fixed lint errors * [MM-289] Fix ci error * Review fixes: Add new types, error state and some refactoring * Fix package-lock.json file * [MM-527] Review fixes * Create a new type * Update package-lock.json file due to ci errors * Fix type errors --------- Co-authored-by: Nityanand Rai <107465508+Nityanand13@users.noreply.github.com> Co-authored-by: ayusht2810 <ayush.thakur@brightscout.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix #27
Backport mattermost-plugin-sample infra code : use go mod, rework makefile to use sub-module, repair coverage.