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

ExApp UI Implementation (Part1) #135

Merged
merged 34 commits into from
Dec 3, 2023
Merged

ExApp UI Implementation (Part1) #135

merged 34 commits into from
Dec 3, 2023

Conversation

bigcat88
Copy link
Member

@bigcat88 bigcat88 commented Nov 28, 2023

This PR adds ability for ExApp to have entry in TopMenu and provide it's own page when clicking on it.

  • OCS API for specifying InitialStates, Scripts, Styles, TopMenu
  • Small corrections and refactoring of code that relates on UI
  • Small bug fixes to other parts of AppAPI, mostly for FileActions Menu.

PR is ready, after merging this, one additional PR will be created that relates to this one.
That PR will cover missing parts:

  • Docs, changelog update
  • FileActions Menu rework(icon specifying)
  • Fixes of stuff that will be found if any, related to UI
  • CI Actions and Makefile adjusting, to keep js/proxy_js folder
  • Fix of Proxying stuff

Merging this PR will allow nc_py_api's PR to be finished on this theme and finish first example for testing.

Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
bigcat88 and others added 4 commits November 29, 2023 14:52
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Andrey Borysenko <andrey18106x@gmail.com>
Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
@bigcat88 bigcat88 mentioned this pull request Nov 30, 2023
3 tasks
@bigcat88 bigcat88 changed the title ExApp UI ExApp UI Implementation Dec 3, 2023
@bigcat88 bigcat88 marked this pull request as ready for review December 3, 2023 11:07
@bigcat88 bigcat88 changed the title ExApp UI Implementation ExApp UI Implementation (Part1) Dec 3, 2023
@bigcat88 bigcat88 merged commit 5027722 into main Dec 3, 2023
27 checks passed
@bigcat88 bigcat88 deleted the menu-entries-ui branch December 3, 2023 13:30
bigcat88 added a commit to cloud-py-api/nc_py_api that referenced this pull request Dec 3, 2023
Reference: cloud-py-api/app_api#135

First part.
* Added API for UI, Example(not finished yet).

Missing:
* Example and description how to use it in docs, and example in example
folder is not fully finished.

---------

Signed-off-by: Alexander Piskun <bigcat88@icloud.com>
Signed-off-by: Andrey Borysenko <andrey18106x@gmail.com>
Co-authored-by: Andrey Borysenko <andrey18106x@gmail.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
bigcat88 added a commit that referenced this pull request Dec 3, 2023
Changes proposed in this PR(first part is here #135):

CI:

* added Oracle CI test, as previous PR created a new error with Oracle
DB due to missing tests
* TopMenu: icon_url -> icon

Bugs:

* do not spam error into log file in **getExAppFileAction**,
**getExAppMenuEntry**
* Oracle fix for FileAction entry
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.

None yet

2 participants