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

Add some handling for ObjectCustomFieldValues to rt-validator #64

Open
wants to merge 3 commits into
base: stable
Choose a base branch
from

Conversation

dmsimard
Copy link

@dmsimard dmsimard commented Sep 1, 2013

There are no real verifications about ObjectCustomFieldValues dependancies and relations. This aims to improve it a little bit. Ref: http://lists.bestpractical.com/pipermail/rt-users/2013-August/081425.html

David Moreau Simard added 2 commits September 1, 2013 11:00
…tCustomFieldValue that no longer exists and delete the transaction.
…ences an ObjectId (Ticket) that is no longer in the database.
@ruz
Copy link
Contributor

ruz commented Sep 1, 2013

Correct direction. Commit messages are too long, use one short line, empty line and description as long as you like, for example "delete OCFV records pointing at not existing object" for 30a401d and describe further in the description.

The following command is your friend, allows to change history:

git checkout -b tmp
git rebase -i 2549ff0^

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.

2 participants