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

Artifactory docs say module is 'jboss7'. #24547

Closed
dragonpaw opened this issue Jun 9, 2015 · 2 comments
Closed

Artifactory docs say module is 'jboss7'. #24547

dragonpaw opened this issue Jun 9, 2015 · 2 comments
Labels
Bug broken, incorrect, or confusing behavior Documentation Relates to Salt documentation fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Milestone
Approved

Comments

@dragonpaw
Copy link

It's a little confusing that the title for the artifactory states module is shown as "salt.states.jboss7". Any chance of changing the docs to show the more understood name of salt.states.artifactory ?
@jfindlay jfindlay added Documentation Relates to Salt documentation Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around P3 Priority 3 labels Jun 11, 2015
@jfindlay
Copy link
Contributor

@dragonpaw, thanks for the report. doc/ref/states/all/salt.states.artifactory.rst seems like it should be renamed to doc/ref/states/all/salt.states.jboss7.rst.

@jfindlay jfindlay added this to the Approved milestone Jun 11, 2015
jacobhammons added a commit to jacobhammons/salt that referenced this issue Jun 12, 2015
@jacobhammons
Doc bug fixes
c7cdd41 
Refs saltstack#24547
Refs saltstack#24275
Refs saltstack#24375
Refs saltstack#24560
Refs saltstack#24163
@jacobhammons jacobhammons mentioned this issue Jun 12, 2015
Merged
@jacobhammons
Copy link
Contributor

This is fixed, http://docs.saltstack.com/en/latest/ref/states/all/salt.states.artifactory.html
@dragonpaw if you think this update is sufficient please close, thanks!

@jacobhammons jacobhammons added the fixed-pls-verify fix is linked, bug author to confirm fix label Jun 18, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Bug broken, incorrect, or confusing behavior Documentation Relates to Salt documentation fixed-pls-verify fix is linked, bug author to confirm fix P3 Priority 3 severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around
Projects
None yet
Milestone
Approved
Development

No branches or pull requests
3 participants
@dragonpaw @jfindlay @jacobhammons