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

Don't show doc actions when they shouldn't be visible #10651

Merged
merged 2 commits into from
Feb 17, 2021
Merged

Conversation

siemiatj
Copy link
Contributor

Looks like this was disabled when I did some quick changes to unblock Petrica on his inline tab widget effort. So this looks like a temporary thing that should be reverted.

d97d319#diff-1cfcb26643fbe64684980f1488db853ee0b5abe32b52125ee175f6e9054bd87aL55

@petrican where can the inline tab be tested to be sure it's not affected ?

@petrican
Copy link
Contributor

Looks like this was disabled when I did some quick changes to unblock Petrica on his inline tab widget effort. So this looks like a temporary thing that should be reverted.

d97d319#diff-1cfcb26643fbe64684980f1488db853ee0b5abe32b52125ee175f6e9054bd87aL55

@petrican where can the inline tab be tested to be sure it's not affected ?

Will check. I see the inlinetab in action on hevuat box but I am having some cors trouble so need to overcome that in order to test.

@petrican
Copy link
Contributor

I see some issues with the build. Will merge the master into it and see if the issue goes away. As I understood from Tobi that should fix it

Copy link
Contributor

@petrican petrican left a comment

Choose a reason for hiding this comment

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

Looks ok. I tested also the inlinetab and I did not spot any issues. Thanks

@petrican
Copy link
Contributor

If the build is successful it can be merged. Thanks

@siemiatj siemiatj merged commit 28c9c00 into master Feb 17, 2021
@siemiatj siemiatj deleted the frontend-10649 branch February 17, 2021 10:04
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.

None yet

2 participants