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

Initial commit for App Switcher docs #5018

Merged
merged 2 commits into from
Oct 7, 2015
Merged

Initial commit for App Switcher docs #5018

merged 2 commits into from
Oct 7, 2015

Conversation

palecur
Copy link

@palecur palecur commented Sep 23, 2015

No description provided.

@palecur palecur mentioned this pull request Sep 23, 2015
8 tasks
@tbragin tbragin self-assigned this Sep 23, 2015
@tbragin
Copy link
Contributor

tbragin commented Sep 23, 2015

@palecur are the latest docs up somewhere?

@palecur
Copy link
Author

palecur commented Sep 23, 2015

Yes; sorry to have left that out. http://kibanapp.divshot.io/kibana-apps
has the rendered doc.

On Wed, Sep 23, 2015 at 9:15 AM, tbragin [email protected] wrote:

@palecur https://github.com/palecur are the latest docs up somewhere?


Reply to this email directly or view it on GitHub
#5018 (comment).

@tbragin
Copy link
Contributor

tbragin commented Sep 25, 2015

Since we already have the "Kibana plugins" section, let's:

  • Move description of the app switcher at the bottom of the "Kibana plugins" section
  • Remove the "Kibana apps" section
  • Rename mentions of "apps" to "plugins"

@palecur
Copy link
Author

palecur commented Sep 26, 2015

Refactored and render updated.

@tbragin
Copy link
Contributor

tbragin commented Sep 26, 2015

The following URL doesn't work any more, is there a new one? http://kibanapp.divshot.io/kibana-apps

@tbragin
Copy link
Contributor

tbragin commented Sep 26, 2015

Looking at Github changes, LGTM.

palecur pushed a commit that referenced this pull request Oct 7, 2015
Initial commit for App Switcher docs
@palecur palecur merged commit 0b32f8f into elastic:master Oct 7, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants