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

Pass view_name to extra_body_script hook #443

Merged
merged 1 commit into from May 3, 2019

Conversation

Projects
None yet
2 participants
@russss
Copy link
Contributor

commented May 2, 2019

At the moment it's not easy to tell whether the hook is being called
in (for example) the row or table view, as in both cases the
database and table parameters are provided.

This passes the view_name added in #441 to the extra_body_script
hook.

@russss russss referenced this pull request May 2, 2019

Open

Release version 1.0 #4

1 of 5 tasks complete
Pass view_name to extra_body_script hook
At the moment it's not easy to tell whether the hook is being called
in (for example) the row or table view, as in both cases the
`database` and `table` parameters are provided.

This passes the `view_name` added in #441 to the `extra_body_script`
hook.

@russss russss force-pushed the russss:view-name-in-extra-body-script branch from e81abfe to 83b6b82 May 2, 2019

@simonw simonw merged commit bf229c9 into simonw:master May 3, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.