-
Notifications
You must be signed in to change notification settings - Fork 261
Tableau Exchange branding #425
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
Conversation
|
Craig I notice the branch conflicts here - this branch might need rebasing again from main, and maybe that'll fix some of those typos like 'the the' which I think you fixed previously. |
typofix
more gallery references
|
@Craigfis what brian said. (John and I just found this PR) Get the conflicts sorted and we'll merge it up! |
|
If only I knew how to rebase my branch in this github web UI... |
|
Thanks for the contribution! Unfortunately we can't verify the commit author(s): Craig Fisher <c***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, refresh the status of this Pull Request. |
|
@Craigfis Hey, Craig. I can fix the branding from Gallery to Exchange. I need to apologize, as I had this on my plate to correct for the v1.8 release. It's probably a lot easier for me to get this change. It looks like you were trying to merge directly with the Here's a new PR to merge with |
|
I'll close this one in favor of Dave's #435 |
No description provided.