Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Core show channels action and minor improvements#274

Open
AlexAlexKud wants to merge 10 commits into
AsterNET:masterAsterNET/AsterNET:masterfrom
AlexAlexKud:CoreShowChannelsImprovementAlexAlexKud/AsterNET:CoreShowChannelsImprovementCopy head branch name to clipboard
Open

Core show channels action and minor improvements#274
AlexAlexKud wants to merge 10 commits into
AsterNET:masterAsterNET/AsterNET:masterfrom
AlexAlexKud:CoreShowChannelsImprovementAlexAlexKud/AsterNET:CoreShowChannelsImprovementCopy head branch name to clipboard

Conversation

@AlexAlexKud

Copy link
Copy Markdown

Better support CoreShowChannelsAction:

MAIN:
AbstractChannelEvent now inherits ResponseEvent;
CoreShowChannelsAction now inherits ManagerActionEvent;
Added events CoreShowChannelEvent and ;CoreShowChannelsCompleteEvent;

BTW:
Added support for TimeSpan and Guid properies in Parsing and ToString methods;
Added AssemblyInfo attributes autogeneration ignore - this gives build an error in VS 2022

EXAMPLE:
Now you can use CoreShowChannelsAction and receives its responses:

var action = new CoreShowChannelsAction();
var responses = manager.SendEventGeneratingAction(action);

AbstractChannelEvent inherits ResponseEvent;
CoreShowChannelsAction inherits ManagerActionEvent;
Added events CoreShowChannelEvent and ;CoreShowChannelsCompleteEvent;

Added support for TimeSpan and Guid properies;
Comment thread Asterisk.2013/Asterisk.NET/Manager/Event/CoreShowChannelEvent.cs Outdated
@Deantwo

Deantwo commented May 21, 2023

Copy link
Copy Markdown
Collaborator

I can see that you changed indention from four spaces ( ) to taps ( ). This makes the diff kinda annoying to read. Any chance you could change that to spaces?

Of course, now that I think about there, there is a an option to hide whitespaces in the diff.
image

But is is annoying to read the code on GitHub when the indention doesn't match.

@AlexAlexKud

AlexAlexKud commented May 22, 2023

Copy link
Copy Markdown
Author

I can see that you changed indention from four spaces ( ) to taps ( ). This makes the diff kinda annoying to read. Any chance you could change that to spaces?

Fair remark! Fixed

@AlexAlexKud

Copy link
Copy Markdown
Author

Comment to get attention
@Deantwo @usender

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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.