Conversation
Merge master to live
Update .openpublishing.publish.config.json
Merge update to live
added "ms.suite": "office",
…tadata-2 Update docfx.json
Merge 4-17 TOC updates to live
Merge nav page updates to live
Merge to live!
Merge to live!
I used the old code ("theCell != null") and got null reference exceptions when cells were empty.
I found that theCell.InnerText.Length>0 avoided the null reference exception. Using Object.ReferenceEquals(), or !(x== null) didn't change the outcome.
Line 419: Null Reference Exception on empty cells
Merge to live!
#### The text following is a meme PR summary. Skip to the end for a plain-english PR summary --- Hey MSFT Dudes, Dudettes, et al... I was looking at the Office Open XML (eXTREMELY MERCILESS LANGUAGE) docs, and while the documentation was like 95% tubular, I had a no-bueno run in with a pair of EVIL broken links... I trench-coated the new urls (dug the new ones up like a trench, then covered for whoever broke them like a coat of paint) and asked you guys for the dankest of pulls to GET THAT CODE IN THERE. Take a look and approve, and this docs page will be totally tubular once more. --- ### SUPER Serious PR Summary At a glance, it looks like dev.office.com is a (newly?) dead subdomain that got migrated into docs.microsoft.com. I updated the URLS to match the new locations for those resources. I'm 95% sure the amended urls match what the old resources were, but if I'm wrong, just know that the links are broken. Thanks for all the awesome stuff you make! Cheers, Rapdash
|
Docs Build status updates of commit 4eeeb8c: ❌ Validation status: errorsPlease follow instructions here which may help to resolve issue.
For more details, please refer to the build report. Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
6 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Kicking off new build to fix build error