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

fix verbiage for Dashboard title #13349

Merged
merged 1 commit into from
Dec 26, 2018
Merged

fix verbiage for Dashboard title #13349

merged 1 commit into from
Dec 26, 2018

Conversation

yashodha
Copy link
Contributor

@yashodha yashodha commented Dec 24, 2018 •
edited
Loading

Overview

fix verbiage for Dashboard title

Before

User Dashboard title is wrongly shown as View Relationship
dashboard_title_before

After

title reads View Contact Dashboard
dashboard_after

Comments

Link when click goes to Dashboard of related contact not the relationship record

@civibot
Copy link

civibot bot commented Dec 24, 2018

(Standard links)

@civibot civibot bot added the master label Dec 24, 2018
jitendrapurohit
jitendrapurohit approved these changes Dec 26, 2018
View reviewed changes
Copy link
Contributor

@jitendrapurohit jitendrapurohit left a comment

Choose a reason for hiding this comment

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

Tested the PR. The link title is shown correctly now.

@yashodha
Copy link
Contributor Author

@jitendrapurohit thanks! merging as per review

@yashodha yashodha merged commit 70c753a into civicrm:master Dec 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jitendrapurohit jitendrapurohit jitendrapurohit approved these changes

Assignees
No one assigned
Labels
master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@yashodha @jitendrapurohit