search

DNNCommunity / Dnn.WebAnalytics

A Dnn (formerly DotNetNuke) module for capturing information about visitors. Includes a report UX and a map UX to show location information.
http://dnncommunity.org/Community/Website-Metrics
10 stars 12 forks source link

Isssues/table renaming #6

Open WillStrohl opened 5 years ago

WillStrohl commented 5 years ago

This renames the tables, table objects, as well as adds support for DNN's DAL2 so it can be installed in any DNN instance

Resolves issue #5

david-poindexter commented 5 years ago

@hismightiness great contribution and thanks for implementing so many best practices in this PR. Can you resolve the one conflict in Controllers/VisitorController.cs? Then we'll be ready to merge.

WillStrohl commented 5 years ago

There wasn't any conflicts when I submitted the pull request. It looks like this one has been created by @mathisjay due to a direct commit to master. Jay, can you please revert your direct commits?

WillStrohl commented 5 years ago

@david-poindexter I helped @mathisjay revert the commits in master. This should be good-to-go again. Let me know. :)

WillStrohl commented 5 years ago

@david-poindexter Don't merge this yet. I think we may have found a necessary update.

WillStrohl commented 5 years ago

Okay... We should be good again, David. 😆

david-poindexter commented 5 years ago

@hismightiness - can you please correct your first comment to the correct GitHub syntax for referencing and autoclosing the issue? See THIS LINK for more information on the correct syntax. Thanks!