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

#2527 Rename the documentation filename from 'visualise_charts_with_plotly.md' to 'visualise_charts_with_kedro_viz.md' #3189

Closed
wants to merge 1 commit into from

Conversation

Gundalai-Batkhuu
Copy link

Description

The issues was created to resolve the issue #2527

Development notes

  • Renamed the documentation markdown file 'visualise_charts_with_plotly.md'
  • Updated the {toctree} in the parent markdown file
  • Tested by running the CI/CD checks locally (make lint, make test, make e2e-tests)

…lotly.md' to 'visualise_charts_with_kedro_viz.md'

- Renamed the documentation markdown file 'visualise_charts_with_plotly.md'
- Updated the {toctree} in the parent markdown file

Signed-off-by: Gundalai Batkhuu <gdbt0623@gmail.com>
@astrojuanlu
Copy link
Member

Sorry @Gundalai-Batkhuu but as @stichbury mentioned, this was a good contribution but it overlaps with kedro-org/kedro-viz#1586 🙏🏽 apologies, and hope you can find another issue to work on!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants