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

Deperate isPlugin and isSingleUser part - 2 #5

Merged
merged 2 commits into from
Jul 13, 2024

Conversation

Rajat-Dabade
Copy link
Contributor

This code remove all the check and validation for isPlugin and isSingleUser.

Stack on: #4

@Rajat-Dabade Rajat-Dabade changed the base branch from main to deperate-store-leve July 2, 2024 10:37
@Rajat-Dabade Rajat-Dabade self-assigned this Jul 2, 2024
@Rajat-Dabade Rajat-Dabade added 2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester labels Jul 5, 2024
@Rajat-Dabade Rajat-Dabade requested a review from marianunez July 5, 2024 06:48
@marianunez marianunez requested a review from mgdelacroix July 10, 2024 01:43
Copy link
Member

@mgdelacroix mgdelacroix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

@Rajat-Dabade Rajat-Dabade merged commit 89cbeca into deperate-store-leve Jul 13, 2024
@Rajat-Dabade Rajat-Dabade deleted the deperate-isplugin-2 branch July 13, 2024 06:12
Rajat-Dabade added a commit that referenced this pull request Jul 13, 2024
* refactor: removed the mattermost auth check

* refactor: removed standalone sql part of code

* refactor: removed not required function from store level

* refactor: removed session check for plugin

* refactor: removed the unnecessary test

* refactor: removed MattermostAuth

* Deperate isPlugin and isSingleUser part - 2 (#5)

* removed remaining singleUser and isPlugin code

* chore: linter fixes
Rajat-Dabade added a commit that referenced this pull request Jul 30, 2024
* refactor: isPlugin for API level

* chore: removed GetContextConn as it is not required

* chore: clean up api.go removed none required function

* chore: removed commented code

* refactor: removed the admin as its no log required

* refactor: removed handlePostTeamRegenerateSignupToken as not used anymore

* refactor: updated make file rules

* refactor: removed isGuest code

* refactor: removed auth login password changepassword code

* chore: linter fixes

* refactor: removed user login register code

* chore: linter fixes

* refactor: removed auth layer

* refactor: removed isSingle user checks

* chore: removed debugging logs

* chore: removed debugging logs

* revert: isGuestUser check

* Deprecate standalone SQL store level (#4)

* refactor: removed the mattermost auth check

* refactor: removed standalone sql part of code

* refactor: removed not required function from store level

* refactor: removed session check for plugin

* refactor: removed the unnecessary test

* refactor: removed MattermostAuth

* Deperate isPlugin and isSingleUser part - 2 (#5)

* removed remaining singleUser and isPlugin code

* chore: linter fixes

* MM-59356: Fixing testcase for webapp and server (#6)

* fix: webapp test cases

* refactor: added rules to makefile and added github workflow for testing

* chore: added ci github workflow

* chore: updated github flow for ci and typos

* chore: typefix

* refactor: added rule to install golangci lint

* chore: to trigger action

* fix: error in ci.yml

* refactor: updated rule

* refactor: fix linter issue

* [MM-59349]: Getting Rid of Mattermost Auth layer (#10)

* refactor: get rid of mattermost auth layer

* chore: removed removeDuplicate from isboardsFromRow

* refactor: changed method to private'

* refactor: private method calls private method

* refactor: get license from server API

* [MM-59824]: Removed website code. (#12)

* MM-59294: Workflow updated and release process (#7)

* refactor: updated workflow and Readme

* chore: added Contributing and security.md

* refactor: updated README and makefile with new release process

* refactor: updated the protected branch name

* refactor: readme

* refactor: updated README

* refactor: removed focalboard references

* refactor: updated README

* From `focalboard` to `board` to generate tar bundle (#14)

* refactor: changed name for generating tar

* refactor: changed from board to boards

* refactor: changed the folder name from board to boards

* fix: ci issue

* chore: ci fix

* chore: ci fix

* minor fix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: Dev Review Requires review by a core committer 3: QA Review Requires review by a QA tester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants