Skip to content

Issue #360 - Support for Table attribute#435

Merged
mikependon merged 1 commit intomikependon:masterfrom
cwaldron:feature/attributes
May 16, 2020
Merged

Issue #360 - Support for Table attribute#435
mikependon merged 1 commit intomikependon:masterfrom
cwaldron:feature/attributes

Conversation

@cwaldron
Copy link
Copy Markdown
Contributor

I got tired of waiting for @lauw to create a PR so I did it myself. Let me know what you think. I added support for TableAttribute, ColumnAttribute, & KeyAttribute using the attributes as defined in System.ComponentModel.Annotations.

Cheers.

@mikependon mikependon self-assigned this May 15, 2020
@mikependon
Copy link
Copy Markdown
Owner

To be honest, I really liked this PR. Big thanks mate! :) Let me make some minor comments and let us collaborate further there.

@mikependon mikependon added the enhancement New feature or request label May 15, 2020
@mikependon mikependon merged commit 5597b8b into mikependon:master May 16, 2020
mikependon added a commit that referenced this pull request May 16, 2020
cwaldron added a commit to cwaldron/RepoDb that referenced this pull request May 16, 2020
cwaldron added a commit to cwaldron/RepoDb that referenced this pull request May 16, 2020
…iting to favor Map & Primary attributes.
@mikependon
Copy link
Copy Markdown
Owner

Hey @cwaldron - it seems that I found a bug here. The usage of Table attribute is Table(TableName, Schema). RepoDb seems to expect it like this Table("[Schema].[TableName]"). Is this something you can handle?

Here is the link where the potential bug is located.

@cwaldron
Copy link
Copy Markdown
Contributor Author

cwaldron commented Jul 8, 2020

Hey @mikependon I just noticed this. Is this still an issue. I'll take a look if it is.

Cheers.

@mikependon
Copy link
Copy Markdown
Owner

@cwaldron - I created a separate case for this and issued a fix. Here is the separate case #464

@cwaldron
Copy link
Copy Markdown
Contributor Author

cwaldron commented Jul 8, 2020

Thanks @mikependon for taking care of this. Great work as always.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants