search

Esri / developer-support

Proof of concept developer code and samples to help be successful with all ArcGIS developer products (Python, NET, JavaScript, Android…). The repository is designed to be an exchange for sharing coding conventions and wisdom to developers at all skill levels.
Apache License 2.0
267 stars 165 forks source link

Add check for Compress Log table #334

Closed kgalliher closed 4 years ago

kgalliher commented 6 years ago

@wellsofthewest - Can you review this? I just added a check for the compress log.

kgalliher commented 5 years ago

@wellsofthewest - Hey there. Can you slap a review on this? All I did was check if the compress log was empty prior to running the script and adding a message that it will be ignored. Thanks!

Update.. I lied. I check the compress log, if it exists I use the last compress date, if not I put in a junk date to avoid a null. It's been a while.

kgalliher commented 5 years ago

@jgravois - It looks like @wellsofthewest doesn't have write access. Can you grant that to him?

jgravois commented 5 years ago

I'd love to, but I don't have đŸ”‘ s to any @esri repos anymore.

email github_admin@esri.com and ask them to add @wellsofthewest to the 'Developer Support' team. (at least I think that's what I named it).

thank you both for continuing to maintain this project!

kgalliher commented 4 years ago

@jgravois - can you approve this? I have a customer I want to share this with but it won't work properly for them without the change in this branch.

jgravois commented 4 years ago

since I'm no longer a member of the organization my approval won't look any different than @wellsofthewest's.

have the admin's still not added him to the @Esri Developer Support team? @phpmaps is this something you can help with?

in the meantime, why not just share the file in your natas333x2-patch-1 branch with the customer?

https://github.com/Esri/developer-support/blob/natas333x2-patch-1/arcsde-sql/sql-server/geodatabase-management/versioning-statistics/versioning-statistics.md

lboyd93 commented 4 years ago

@kgalliher @jgravois @wellsofthewest I am a maintainer of this repo now since we are getting the Support Analysts back into contributing. I will go ahead and approve of this! Thanks. :)

jgravois commented 4 years ago

awesome. thanks for jumping in @lboyd93!