Skip to content

Change dashboard(slug) to dashboard(id)#126

Open
DanaMiller-PGH wants to merge 1 commit into
getredash:masterfrom
DanaMiller-PGH:master
Open

Change dashboard(slug) to dashboard(id)#126
DanaMiller-PGH wants to merge 1 commit into
getredash:masterfrom
DanaMiller-PGH:master

Conversation

@DanaMiller-PGH

Copy link
Copy Markdown

addresses: #125

@justinclift

Copy link
Copy Markdown
Member

Interesting. Looks like we need some kind of proper test suite for this redash-toolbelt repo too.

I'm happy to merge this once the "restyled" test is passing. If you're ok to add the commit that it's suggesting in the error message, that should do the trick (in theory):

https://restyled.io/gh/getredash/repos/redash-toolbelt/jobs/3452846

@DanaMiller-PGH

Copy link
Copy Markdown
Author

@justinclift Sorry it's taken me so long to circle back on this. The restyling output seems to be gone. I'll try a new commit to trigger again.

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