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

REST API to delete dashboard #3671

Closed
wants to merge 1 commit into from
Closed

Conversation

dairyo
Copy link
Contributor

@dairyo dairyo commented Apr 2, 2019

What type of PR is this? (check all applicable)

  • Feature

Description

My system wants to remove unused dashboards.

Related Tickets & Documents

--

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

--

@arikfr
Copy link
Member

arikfr commented Apr 2, 2019

Hi,

It's great you have these needs and we appreciate you want to contribute back to the project. But if you're adding new functionality, it's best to discuss it first. You seem to have several needs you're trying to address, so it's best if you take a moment and open a thread on https://discuss.redash.io/c/development to discuss them before moving forward.

I'm not sure if you all your requests are applicable to the project, but what doesn't fit you can look into moving into an extension that you will deploy in your own setup.

@dairyo
Copy link
Contributor Author

dairyo commented Apr 2, 2019

@arikfr
Thank you for the comment. I reconsider that whether PRs can contribute this project.

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.

2 participants