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

jsconsole: dynamically load method signatures from all classes implementing BaseScopeable #220

Closed
wants to merge 4 commits into from

Conversation

hi-ko
Copy link
Contributor

@hi-ko hi-ko commented Mar 11, 2024

CHECKLIST

  • There aren't existing pull requests attempting to address the issue mentioned here
  • Submission developed in a feature branch--not master

currently jsconsole only supports pregenerated or manually created tern files to support autocomplete.
I added @AFaust 's Webscript AlfrescoScriptAPITernGet.java from his fork to dynamically generate a tern document, so any custom Alfresco Addon would support autocomplete in jsconsole without providin handcrafted tern files

The original webscript has been extended to also handle array properties

RELATED INFORMATION

hi-ko and others added 4 commits October 24, 2022 12:44
* fixed broken links in jsconsole doc menue by using `/latest/` (instead of community)
* added doc links for "5.0 JavaScript Services API", "5.0 JavaScript API", "JavaScript API Cookbook" in old Alfresco wiki
@AFaust AFaust added this to the 1.3.0.0 milestone Mar 29, 2024
@AFaust AFaust self-assigned this Mar 29, 2024
@shazada
Copy link

shazada commented Jun 28, 2024

We're currently testing this PR and will report back when it's working successfully

@ContezzaSup
Copy link

We're currently testing this PR and will report back when it's working successfully

We have tested this PR and at this moment it's not working for us, maybe because it's not based on 1.2.2.0 but has some old code. It generates the json file perfectly but the autocomplete doesn't work, not with the json file nor with the standard autocomplete like search.. However the dynamical generation of the json is very interesting for our usecase.

@AFaust AFaust mentioned this pull request Sep 12, 2024
1 task
@AFaust
Copy link
Contributor

AFaust commented Sep 12, 2024

Since I could not update the source branch to rebase the code to the latest OOTBee Support Tools master, I created a new PR from a branch I pushed to my own GitHub fork. In this branch I also applied some minor cleanup + fixes, like removing the width constraint on the tern hints in the UI, and updated the project version to the 1.2.3.0-SNAPSHOT development version.
I will create a couple of follow-up issue(s) that deal(s) with configuration for the dynamic tern generation, which includes moving some of the documentation texts from the static tern JSON files (kept for the time being, albeit without being used) into the properties file for the tern generation.

@AFaust
Copy link
Contributor

AFaust commented Sep 12, 2024

The PR / changes have been tested to work with ACS 6.1 and 23.1 as well.

@AFaust AFaust closed this Sep 12, 2024
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.

4 participants