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

added ambiguous column description #81

Open
wants to merge 8 commits into
base: current/blead
Choose a base branch
from

Conversation

m0veax
Copy link

@m0veax m0veax commented Jul 17, 2015

My first pull request to the Project. I hope it fit your needs.

I've added a description how to solve Problems with ambiguous columns in ResultSets.

If something don't fit your needs, just let me a note. I will refactor my changes then.

@m0veax
Copy link
Author

m0veax commented Jul 20, 2015

Hi @ribasushi, thanks for commenting my commit.

I've added some error messages, a short description and an example how to use the function.
Additionaly i asked in the channel for more error messages, if some comes up, i will add them.

I hope with my changes it now fits the needs of the documentation.

@ribasushi
Copy link
Collaborator

I am traveling this and next week, sorry for the slower than usual response. This will be reviewed/merged by end of next week.

Cheers!

@karenetheridge
Copy link
Contributor

?

@m0veax
Copy link
Author

m0veax commented Jun 11, 2020

Ah, sry. I thought that was completed and in my fork.

Cleaning up my issues at the moment. IDK if that commits made it to master.

@ribasushi
Copy link
Collaborator

Not just yet, but would not have been lost either way as I track PRs outside of github. Reopening either way.

@ribasushi ribasushi reopened this Jun 12, 2020
lib/DBIx/Class/Manual/Troubleshooting.pod Outdated Show resolved Hide resolved
lib/DBIx/Class/Manual/Troubleshooting.pod Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants