-
Notifications
You must be signed in to change notification settings - Fork 33
Updated Accumulation Chart UG documentation #3093
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
Conversation
Build Status: INPROGRESS 🔃 |
CI Status: FAILURE ❌ |
Build Status: INQUEUE 🕒 |
The running CI Job is terminated due to changes committed on the source branch for this Merge Request and CI triggered for latest commit. |
CI Status: ABORTED ❌ |
Build Status: INQUEUE 🕒 |
Build Status: INPROGRESS 🔃 |
CI Status: FAILURE ❌ |
Build Status: INPROGRESS 🔃 |
CI Status: SUCCESS ✅ |
Build Status: INPROGRESS 🔃 |
CI Status: FAILURE ❌ |
Build Status: INPROGRESS 🔃 |
CI Status: FAILURE ❌ |
Build Status: INPROGRESS 🔃 |
CI Status: FAILURE ❌ |
Build Status: INQUEUE 🕒 |
Build Status: INPROGRESS 🔃 |
CI Status: FAILURE ❌ |
Build Status: INPROGRESS 🔃 |
CI Status: SUCCESS ✅ |
{% highlight c# tabtitle="Annotation.cs" %} | ||
{% include code-snippet/chart/accumulation-charts/annotation/annotation/annotation.cs %} | ||
{% highlight c# tabtitle="CSHTML.cs" %} | ||
... |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Share reason to change inline code block
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @rajendranr-5483 ,
Because, Annotation.cs class is used in both platforms (Core and MVC). I checked only core platform. So, i added code snippet inside here.
Also, some codebehind file is mentioned like below,
var opt=new Games().GamesList(); --> in documentation only mentioned this. Not include data inside this method.
Regards,
@Backiaraj
public class PieChartData | ||
{ | ||
public string xValue; | ||
public double yValue; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
variable casing is incorrect
Closed previous changes in old hotfix. Refer this.
I modified the ASP.NET CORE components UG documentation and code snippet also. The component list is given below,