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.

Sign up for GitHub

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

Jump to bottom

Add option in ATDPApplication to hide the provenance menu (aka Session Analysis Management) #428

Merged
merged 3 commits into from
Oct 1, 2020
Merged

Add option in ATDPApplication to hide the provenance menu (aka Session Analysis Management) #428

merged 3 commits into from
Oct 1, 2020

Conversation

oltionchampari
Copy link
Contributor

@oltionchampari oltionchampari commented Sep 29, 2020 •
edited by thinkh
Loading

Summary

  • Added in ATDPApplication an option showProvenanceMenu to hide the provenance menu.

@oltionchampari oltionchampari added type: feature New feature or request release: minor PR merge results in a new minor version labels Sep 29, 2020
thinkh
thinkh approved these changes Oct 1, 2020
View reviewed changes
Copy link
Member

@thinkh thinkh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works for me! 👍

@thinkh thinkh changed the title Add option in ATDPApplication to hide the provenance menu Add option in ATDPApplication to hide the provenance menu (aka Session Analysis Management) Oct 1, 2020
@thinkh thinkh merged commit 6e6c47e into develop Oct 1, 2020
@thinkh thinkh deleted the ochampari/hide_session_managment_option branch October 1, 2020 11:17
@thinkh thinkh mentioned this pull request Oct 1, 2020
Merged
oltionchampari added a commit that referenced this pull request Oct 1, 2020
@oltionchampari
Merge pull request #431 from datavisyn/ochampari/hide_session_managme…
2b9c53b 
…nt_option

Set missing default value for `showProvenanceMenu` (amendment for PR #428)
@dvvanessastoiber dvvanessastoiber mentioned this pull request Jan 26, 2021
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thinkh thinkh thinkh approved these changes

@puehringer puehringer Awaiting requested review from puehringer

@lehnerchristian lehnerchristian Awaiting requested review from lehnerchristian

Assignees

@thinkh thinkh

@puehringer puehringer

Labels
release: minor PR merge results in a new minor version type: feature New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@oltionchampari @thinkh @puehringer