Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Merge back the updates previously made #87

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

Closed
wants to merge 60 commits into from

Conversation

hangie
Copy link

@hangie hangie commented Mar 21, 2019

Hi,

Sorry for the long absence. I had been caught up and couldn't check the changes. I've now merged with the latest master and replaced (mostly alternative) the previous UI implementation with a more complete version that supports all components.
I've also added INI file for configuration.
Fixed a number of bugs (I'll have to go through them later and cross ref them).
Added the XML export of data to fix some bugs.
Export queries as .sql when exporting as SQL and as .bas when not.
Have a look. Pretty sure this is good to go.

Cheers,
Mark.

hangie added 30 commits March 17, 2018 10:28
Updates from master repo
@hangie
Copy link
Author

hangie commented Mar 21, 2019

This fixes #84, #83, #79, #50, #61

@hangie
Copy link
Author

hangie commented Mar 22, 2019

Fixes #85, #57.

@timabell
Copy link
Member

timabell commented Mar 25, 2019

Nice work, thanks for the contribution. I'm afraid I'm going to be a bit picky, it's difficult to see in this PR what has been done as it has many commits and merges. If there are problems it'll be impossible to work out the source of an issue by looking at the history.

Please break this up into one PR per feature / bugfix, and make sure each one is based on timabell/master

Something like:

o  timabell/master
  \_ o _ o export-as-sql
  \_ o _ o fix-backslash-escaping

@hangie
Copy link
Author

hangie commented Mar 26, 2019

This will be a bit of a challenge, but I'll do what I can.

Added need pull request to cover fix for previous changes in merge/hangie_develop.

Will follow this with other changes.

@hangie hangie closed this Mar 26, 2019
@timabell
Copy link
Member

#88

hecon5 pushed a commit to hecon5/msaccess-vcs-integration that referenced this pull request Jun 15, 2021
Started building out functions for handling translation of strings. See msaccess-vcs-integration#87
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