Skip to content
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

Replace all instances of "Gitlab" with "GitLab" #31

Closed
jasonblais opened this issue May 6, 2019 · 0 comments · Fixed by #32
Closed

Replace all instances of "Gitlab" with "GitLab" #31

jasonblais opened this issue May 6, 2019 · 0 comments · Fixed by #32
Assignees

Comments

@jasonblais
Copy link
Contributor

I submitted a PR that updates the strings in plugins.json, but there may be other occurrences of it https://github.com/manland/mattermost-plugin-gitlab/pull/30

@manland manland self-assigned this May 7, 2019
raghavaggarwal2308 added a commit that referenced this issue 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
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants