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

dev/core#2660 - Remove old menu entry for "New Tag" #20686

Merged
merged 1 commit into from
Jun 22, 2021

Conversation

demeritcowboy
Copy link
Contributor

Overview

https://lab.civicrm.org/dev/core/-/issues/2660

Before

  1. Contact -> New Tag.
  2. It goes to the same place as Manage Tags.

After

No practical change.

Technical Details

This seems to be left over from when the tags screen was redone. There is a real "new tag" url, but it's a different url and on the manage tags screen you have more options like parents or tagsets, so since people are already going to that screen in practice they aren't losing anything.

Comments

Only for new installs.

@civibot
Copy link

civibot bot commented Jun 21, 2021

(Standard links)

@colemanw
Copy link
Member

Looks good @demeritcowboy
On an administrative note, you can have this PR automatically close the issue by putting "Fixes dev/core#2660" in the commit message (not the PR, has to be the actual commit msg)

@demeritcowboy
Copy link
Contributor Author

Ok thanks noted - although I'd still want to go put in the milestone. It's probably not simple to have it do that automatically.

@demeritcowboy
Copy link
Contributor Author

Interesting - there's only one record in the civicrm_contact table. Hmm.

@demeritcowboy
Copy link
Contributor Author

I see what it is. The regen script needs updating, for fantastic reasons which the margins here are too small to contain.

@seamuslee001 seamuslee001 merged commit a7e52eb into civicrm:master Jun 22, 2021
@colemanw
Copy link
Member

Goodbye old new tag

@demeritcowboy demeritcowboy deleted the newtag branch June 22, 2021 12:36
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.

3 participants