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

Rename index_pattern_management plugin (#37) #132

Merged
merged 3 commits into from
Mar 12, 2021
Merged

Rename index_pattern_management plugin (#37) #132

merged 3 commits into from
Mar 12, 2021

Conversation

boktorbb
Copy link
Contributor

@boktorbb boktorbb commented Mar 9, 2021

Signed-off-by: Bishoy Boktor [email protected]

Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

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

LGTM !!

Signed-off-by: Bishoy Boktor <[email protected]>
Signed-off-by: Bishoy Boktor <[email protected]>
@boktorbb boktorbb requested a review from kavilla March 12, 2021 01:47
@boktorbb boktorbb dismissed kavilla’s stale review March 12, 2021 01:47

changes made

Copy link
Member

@kavilla kavilla left a comment

Choose a reason for hiding this comment

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

Responded to the update. Im good with just moving forward and updating the logic later as it only relates to tests. I just noted it down for later.

LGTM! But I will let you make the final call on merging! Thanks!

@boktorbb
Copy link
Contributor Author

We can note the logic needs to change and work on it during build phase

@boktorbb boktorbb merged commit 5915faa into opensearch-project:dashboards Mar 12, 2021
kavilla referenced this pull request in kavilla/OpenSearch-Dashboards Mar 19, 2021
* Rename index_pattern_management plugin (#37)

Signed-off-by: Bishoy Boktor <[email protected]>

* Address PR comments

Signed-off-by: Bishoy Boktor <[email protected]>

* Address PR comments

Signed-off-by: Bishoy Boktor <[email protected]>
kavilla pushed a commit that referenced this pull request Mar 20, 2021
* Rename index_pattern_management plugin (#37)

Signed-off-by: Bishoy Boktor <[email protected]>

* Address PR comments

Signed-off-by: Bishoy Boktor <[email protected]>

* Address PR comments

Signed-off-by: Bishoy Boktor <[email protected]>
mihirsoni pushed a commit that referenced this pull request Mar 20, 2021
* Rename index_pattern_management plugin (#37)

Signed-off-by: Bishoy Boktor <[email protected]>

* Address PR comments

Signed-off-by: Bishoy Boktor <[email protected]>

* Address PR comments

Signed-off-by: Bishoy Boktor <[email protected]>
gaobinlong pushed a commit to gaobinlong/OpenSearch-Dashboards that referenced this pull request Sep 11, 2023
* fix: workspace permission entire update

Signed-off-by: Lin Wang <[email protected]>

* refactor: rename to convertToFullWorkspacePermissions and remove read and write permission

Signed-off-by: Lin Wang <[email protected]>

* Revert "refactor: rename to convertToFullWorkspacePermissions and remove read and write permission"

This reverts commit a4b7ba7.

Signed-off-by: Lin Wang <[email protected]>

* Revert "fix: workspace permission entire update"

This reverts commit d972e13.

Signed-off-by: Lin Wang <[email protected]>

* fix: update to client.create to override entire object

Signed-off-by: Lin Wang <[email protected]>

* feat: add workspace version limit for object update

Signed-off-by: Lin Wang <[email protected]>

---------

Signed-off-by: Lin Wang <[email protected]>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Sep 14, 2023
…nsearch-project#141)

* fix: workspace permission entire update



* refactor: rename to convertToFullWorkspacePermissions and remove read and write permission



* Revert "refactor: rename to convertToFullWorkspacePermissions and remove read and write permission"

This reverts commit a4b7ba7.



* Revert "fix: workspace permission entire update"

This reverts commit d972e13.



* fix: update to client.create to override entire object



* feat: add workspace version limit for object update



---------


(cherry picked from commit 0bd03df)

Signed-off-by: Lin Wang <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Hailong-am pushed a commit to Hailong-am/OpenSearch-Dashboards that referenced this pull request Sep 15, 2023
* fix: workspace permission entire update

Signed-off-by: Lin Wang <[email protected]>

* refactor: rename to convertToFullWorkspacePermissions and remove read and write permission

Signed-off-by: Lin Wang <[email protected]>

* Revert "refactor: rename to convertToFullWorkspacePermissions and remove read and write permission"

This reverts commit a4b7ba7.

Signed-off-by: Lin Wang <[email protected]>

* Revert "fix: workspace permission entire update"

This reverts commit d972e13.

Signed-off-by: Lin Wang <[email protected]>

* fix: update to client.create to override entire object

Signed-off-by: Lin Wang <[email protected]>

* feat: add workspace version limit for object update

Signed-off-by: Lin Wang <[email protected]>

---------

Signed-off-by: Lin Wang <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants