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.

Sign up for GitHub

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

Jump to bottom

feat: Split privilege resource / data source in resource / data source for each type #372

Merged
merged 36 commits into from
Sep 21, 2023
Merged

feat: Split privilege resource / data source in resource / data source for each type #372

merged 36 commits into from
Sep 21, 2023

Conversation

fabrue
Copy link
Member

@fabrue fabrue commented Aug 15, 2023

No description provided.

@anmoel anmoel changed the title feat: use new nexus client version feat: Split privilege resource and data source in resource / data source for each type Sep 20, 2023
@anmoel anmoel changed the title feat: Split privilege resource and data source in resource / data source for each type feat: Split privilege resource / data source in resource / data source for each type Sep 20, 2023
fabrue and others added 27 commits September 20, 2023 17:39
@fabrue @chris-ruecker
feat: use new nexus client version
99b458f
@fabrue @chris-ruecker
feat: use most recent nexus client version
6ef1c29
@fabrue @chris-ruecker
feat: add nexus_privilege_application resource
6971b8b
@chris-ruecker
docs: Add name in priv_application example
a0814e0
@chris-ruecker
chore: Update go version to 1.21
67c710c
@chris-ruecker
feat: Add nexus_privilege_script resource
8c85c06
@chris-ruecker
refactor: rename variable
70fad27
@chris-ruecker
fix: Remove invalid actions from script priv
059525e
@chris-ruecker
refactor: Add tool for formatting actions
e8d1f61
@chris-ruecker
feat: Add privilege_repository_view resource
f2ae22a
@chris-ruecker
feat: add privilege_repository_admin resource
23b04ad
@chris-ruecker
feat: add privilege repository_content_selector
3a770bd
@chris-ruecker
docs: fix resource name in example
92d38ee
@chris-ruecker
feat: add privilege_wildcard resource
ff2da38
@chris-ruecker
feat: add privilege_wildcard data source
a476ac5
@chris-ruecker
feat: add privilege_repository_view datasource
2d7f25f
@chris-ruecker
feat: add privilege_repository_admin datasource
d1190d8
@chris-ruecker
feat: add privilege content selector datasource
da7b87d
@chris-ruecker
feat: add privilege_script datasource
b361807
@chris-ruecker
test: Test individual actions in privileges
c08b046
@chris-ruecker
fix: Test and correct actions for app priv
180ca71
@chris-ruecker
docs: unify resource names in privilege examples
4193f39
@chris-ruecker
fix: mark domain as required privilege_application
791276d
@chris-ruecker
docs: generate docs for privilege res/ds
50de11d
@chris-ruecker
refactor: Replace ioutil.ReadFile with os.ReadFile
2a064c6
@chris-ruecker
refactor: remove deprecated use of plugin.Debug
3efa507
@chris-ruecker
refactor: remove unnecessary import
97da3b0
@anmoel anmoel marked this pull request as ready for review September 20, 2023 15:42
@anmoel
Copy link
Member

anmoel commented Sep 20, 2023 •
edited
Loading

fix #291 #292 #293 #294 #295 #296

anmoel
anmoel requested changes Sep 21, 2023
View reviewed changes
examples/resources/nexus_privilege_repository_admin/resource.tf Outdated Show resolved Hide resolved
examples/resources/nexus_privilege_repository_content_selector/resource.tf Outdated Show resolved Hide resolved
examples/resources/nexus_privilege_repository_view/resource.tf Outdated Show resolved Hide resolved
examples/resources/nexus_privilege_script/resource.tf Outdated Show resolved Hide resolved
internal/services/security/data_source_privilege_repository_admin.go Outdated Show resolved Hide resolved
internal/services/security/data_source_privilege_script.go Outdated Show resolved Hide resolved
internal/services/security/data_source_privilege_repository_view.go Outdated Show resolved Hide resolved
internal/services/security/data_source_privilege_repository_content_selector.go Outdated Show resolved Hide resolved
anmoel
anmoel approved these changes Sep 21, 2023
View reviewed changes
Copy link
Member

@anmoel anmoel left a comment

Choose a reason for hiding this comment

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

lgtm

@anmoel anmoel merged commit 5dddfec into main Sep 21, 2023
3 checks passed
@anmoel anmoel deleted the feat/privilege-resources branch September 21, 2023 12:33
@anmoel
Copy link
Member

anmoel commented Sep 21, 2023

belongs to #291 #292

This was linked to issues Sep 21, 2023
Resource and data source nexus_privilege_repository_view #291
Closed
Resource and data source nexus_privilege_script #292
Closed
Resource and data source nexus_privilege_wildcard #293
Closed
Resource and data source nexus_privilege_repository_admin #294
Closed
Resource and data source nexus_privilege_repository_content_selector #295
Closed
Resource and data source nexus_privilege_application #296
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anmoel anmoel anmoel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@fabrue @anmoel @chris-ruecker