Skip to content

(maint) Remove redirect to PuppetDB dashboard#1380

Closed
ajroetker wants to merge 1 commit intopuppetlabs:masterfrom
ajroetker:maint/remove_redirect_to_dashboard
Closed

(maint) Remove redirect to PuppetDB dashboard#1380
ajroetker wants to merge 1 commit intopuppetlabs:masterfrom
ajroetker:maint/remove_redirect_to_dashboard

Conversation

@ajroetker
Copy link
Contributor

This commit changes the PuppetDB dashboard to be served as a file
response on "/" explicitly instead of a redirect to
"/dashboard/index.html". This allows us to refrain from passing
url-prefix to the main PuppetDB app routes function, as well as
modularize the dashboard to its own handler so it can be served
separately from the PuppetDB query app.

This commit changes the PuppetDB dashboard to be served as a file
response on "/" explicitly instead of a redirect to
"/dashboard/index.html". This allows us to refrain from passing
url-prefix to the main PuppetDB app `routes` function, as well as
modularizing the dashboard to its own handler so it can be served
separately from the PuppetDB query app.
@ajroetker
Copy link
Contributor Author

Just something that occurred to me might be nice for our dashboard while going through an om tutorial.

@pljenkinsro
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1165/

@ajroetker ajroetker added the work in progress (...and please don't merge) label May 6, 2015
@ajroetker ajroetker closed this May 6, 2015
@ajroetker
Copy link
Contributor Author

Closing as it complicates existing work

@ajroetker ajroetker deleted the maint/remove_redirect_to_dashboard branch May 6, 2015 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

work in progress (...and please don't merge)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants