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

Update popover function call method signature #447

Merged
merged 4 commits into from
Aug 13, 2018
Merged

Update popover function call method signature #447

merged 4 commits into from
Aug 13, 2018

Conversation

izshreyansh
Copy link
Contributor

Currently on line #275
$('a[rel=popover]').popover({ html: 'true',

html true where 'true' is within quotes, Which causes issue when trying to open popover. As popover expects Boolean. As mentioned at: https://getbootstrap.com/docs/4.0/components/popovers/#options

Currently on line #275
`$('a[rel=popover]').popover({ html: 'true',`

html true where `'true'` is within quotes, Which causes issue when trying to open popover. As popover expects Boolean. As mentioned at: https://github.com/akaunting/akaunting/issues/url
@cuneytsenturk
Copy link
Member

cuneytsenturk commented Aug 8, 2018

Thanks @shreyansh9226

Could you fix those pages Bill, Invoice create and edit template,

I'll merge if you send those changes.

Fixed: Calling popover with html variable under quotes.
Fixed: Calling popover with html variable under quotes.
Fixed: Calling popover with html variable under quotes.
@izshreyansh
Copy link
Contributor Author

izshreyansh commented Aug 13, 2018

@cuneytsenturk Hi, Sorry for the delay. But its done.

@izshreyansh izshreyansh reopened this Aug 13, 2018
@cuneytsenturk cuneytsenturk added this to the 1.2.13 milestone Aug 13, 2018
@cuneytsenturk cuneytsenturk merged commit 784d2fd into akaunting:master Aug 13, 2018
@cuneytsenturk
Copy link
Member

Thank you @shreyansh9226

ihsanberahim pushed a commit to ihsanberahim/akaunting that referenced this pull request Jul 3, 2023
Bumps [composer/composer](https://github.com/composer/composer) from 2.0.11 to 2.0.13.
- [Release notes](https://github.com/composer/composer/releases)
- [Changelog](https://github.com/composer/composer/blob/master/CHANGELOG.md)
- [Commits](composer/composer@2.0.11...2.0.13)

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

None yet

2 participants