[jira] [Commented] (GEODE-7282) Geode Native Client .NET guide: Fix capitalization issues

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/GEODE-7282?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17408030#comment-17408030 ] 

ASF GitHub Bot commented on GEODE-7282:
---------------------------------------

davebarnes97 opened a new pull request #863:
URL: https://github.com/apache/geode-native/pull/863


   Update method names to follow C# capitalization conventions.
   Addresses occurrences enumerated in the JIRA ticket - not a comprehensive survey of the entire user guide.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Geode Native Client .NET guide: Fix capitalization issues
> ---------------------------------------------------------
>
>                 Key: GEODE-7282
>                 URL: https://issues.apache.org/jira/browse/GEODE-7282
>             Project: Geode
>          Issue Type: Bug
>          Components: docs, native client
>            Reporter: Dave Barnes
>            Assignee: Dave Barnes
>            Priority: Major
>              Labels: pull-request-available
>
> Reviewer [~karensmolermiller] approved the Native Client manual reorganization (GEODE-7262), but noted the following issues with the .NET version. These need cleaning up.
> geode-native/dotnet/111/regions/registering-interest-for-entries.html
> - C++ capitalization in 3rd paragraph and in Client API for Registering Interest
> subsection
> - I believe that all the examples use C++ capitalization
>  
> geode-native/dotnet/111/serialization/data-serialization.html
> - The .NET Example subsection has capitalization wrong for the field name
> description. I think that the example was wrong before these docs were
> separated, so this is not a mistake in separation.
>  
> geode-native/dotnet/111/remote-queries.html
> - Step 3 under Executing a Query from the Client should have a capital 'E'
> in Query.Execute(). I think that the example was wrong before these docs were
> separated, so this is not a mistake in separation.
>  
> geode-native/dotnet/111/continuous-queries.html
> - Prose within step 2 of Typical Continuous Query Lifecycle should change
> to CqQuery Execute* methods (in 2 spots).  Original prose only used the
> C++ capitalization.  Step 4 changes to CqQuery.Close.
> - Prose within step 3 of Executing a Continuous Query from the Client
> should change to CqQuery Execute* methods.
>  
> geode-native/dotnet/111/function-execution.html
> - This whole section was originally written with C++ capitalization. I suggest
> that we introduce a separate task/PR to correct the capitalization for the
> .NET docs.
>  
> geode-native/dotnet/111/transactions.html
> - first paragraph of Native Client Transaction APIs should capitalize Begin,
> Commit, and Rollback, as they are function names.
> - second paragraph of Running Native Client Transaction should capitalize Begin,
> Commit, and Rollback, as they are function names.
> - There's a boo-boo in capitalization of the .NET Example. Within a foreach
> code block, it should be
>     var value = GetValueFromExternalSystem();
> because function name start with a capital letter.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)