Skip to content
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

[Applications.Cion] Fix namespace, ServerBase.DisplayName property, docfx #3553

Merged
merged 20 commits into from
Oct 13, 2021

Conversation

jeremy-jang
Copy link
Contributor

@jeremy-jang jeremy-jang commented Sep 13, 2021

Description of Change

Fix ServerBase.DisplayName property and fix doxygen comments

API Changes

@github-actions github-actions bot added the API9 Platform: Tizen 6.5 / TFM : tizen90 label Sep 13, 2021
@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

6 similar comments
@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 173, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 175, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

3 similar comments
@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 175, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 175, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 175, Removed: 11, Changed: 6

Internal API Changed

Added: 33, Removed: 154, Changed: 2

@WonyoungChoi
Copy link
Member

Please rebase with master branch again.

@jeremy-jang
Copy link
Contributor Author

rebased

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.

Added: 0, Removed: 0, Changed: 1

Changed

+ /// <privilege>http://tizen.org/privilege/d2d.datasharing</privilege
/// <since_tizen>9</since_tizen
System.Void Tizen.Applications.ClientBase::TryDiscovery()

@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.

Added: 0, Removed: 0, Changed: 1

Changed

+ /// <privilege>http://tizen.org/privilege/d2d.datasharing</privilege
/// <since_tizen>9</since_tizen
System.Void Tizen.Applications.ClientBase::TryDiscovery()

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Add missing description of exceptions.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Add missing setting _displayName.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
- Remove debugging logs.
- Change log tag to "Tizen.Applications.Cion".

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
- Fix bad codes.
- Update native API changes.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
ServerBase.OnConnentionRequest() -> ServerBase.OnConnectionRequest()

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Fix namespace to Tizen.Applications.Cion from Tizen.Applications.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.
Added: 101, Removed: 101, Changed: 0

@jeremy-jang jeremy-jang changed the title [Applications.Cion] Fix ServerBase.DisplayName property and fix doxygen comments [Applications.Cion] Fix namespace, ServerBase.DisplayName property, docfx Oct 12, 2021
@WonyoungChoi WonyoungChoi merged commit d63174d into Samsung:master Oct 13, 2021
@jeremy-jang jeremy-jang deleted the acr branch October 13, 2021 08:38
dongsug-song pushed a commit that referenced this pull request Nov 9, 2021
…ocfx (#3553)

* [Applications.Cion] Fix DisplayName property

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* [Applications.Cion] Fix doxygen comments

Add missing description of exceptions.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix ServerBase.DisplayName property

Add missing setting _displayName.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add privilege description at ClientBase.Discovery

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add some remarks FilePayload class

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add exception description of Client.TryDiscovery

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add exception description at FilePayload class

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add privlevel descriptions

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix typo

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix logging

- Remove debugging logs.
- Change log tag to "Tizen.Applications.Cion".

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add remarks exceptions at SecurityInfo class

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix native error code

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix interop delegate name

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix cion

- Fix bad codes.
- Update native API changes.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix typo

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix bug of ClientBase.SendPayloadAsync

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Adjust changes of native APIs

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix typo

ServerBase.OnConnentionRequest() -> ServerBase.OnConnectionRequest()

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Add missing privilege requirements

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>

* Fix namespace

Fix namespace to Tizen.Applications.Cion from Tizen.Applications.

Signed-off-by: Sangyoon Jang <jeremy.jang@samsung.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ACR Required API9 Platform: Tizen 6.5 / TFM : tizen90
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants