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

Transition to TS samples #69

Closed
dfenerski opened this issue Aug 3, 2023 · 3 comments
Closed

Transition to TS samples #69

dfenerski opened this issue Aug 3, 2023 · 3 comments
Assignees
Labels
contribution Valuable Contribution in-discussion This item is being discussed internally

Comments

@dfenerski
Copy link

Issue description

The official endorsement of TypeScript & the drop of the beta for the support for it are great things for the framework and seem to set a positive direction for the future of the technology.

However during a recent visit to the documentation page (and also to the SDK in general) it came to me that all the samples in both the docs & in the mini apps in the 'samples' section still very much use var _aMyCustomEnum = ['Foo', 'Bar'] type of code.

While providing a general idea, this style of code is not only old but also contradicts the best practices encouraged for both TS & modern ES development, which may mislead or confuse newcomers - videos, books, blogposts all encourage modern ES use while official docs/samples have not been migrated (rewritten).

Therefore I thought it might be good idea to at least ask what the status & plans on possible migration of such samples are?

Are PR's with sample rewrites to be accepted? If yes this could help a bit the core maintainers with the migration effort.

Sorry if this has been clarified somewhere & I am missing it, otherwise It'd be nice to know.

Thanks & BR

Dimitar

Feedback Type (Optional)

clarity

Page Title on SAP Help Portal (prefilled)

No response

Page URL on SAP Help Portal (prefilled)

No response

@KvM2
Copy link
Contributor

KvM2 commented Aug 3, 2023

Hi @dfenerski , thanks for your contribution and thorough look at the documentation. We appreciate it and will look into this.

@OlMue OlMue added contribution Valuable Contribution in-discussion This item is being discussed internally labels Nov 22, 2023
Copy link

sap-doc-bot bot commented Nov 22, 2023

Thank you for your valuable feedback contribution, @dfenerski! So that we can recognize your contribution in SAP Community, please tell us your SAP Community profile URL in a reply to this comment; don't include any other text, just the URL on its own, like this:

https://people.sap.com/your-user-name

Thanks!

@OlMue
Copy link
Contributor

OlMue commented Nov 22, 2023

Hi Dimitar, sorry for the long silence and thanks again for bringing this up. We're currently updating the documentation (including the tutorials) to bring it up to modern ES standards and also put more and more emphasis on TypeScript. Revamping the samples is also under discussion, but I can't give you an estimate as to when this will be done.

About your question concerning community contributions: I passed this on to the development colleagues, but there hasn't been a decision yet as to this. If you want to address this again directly to them, I suggest that you open another issue with your suggestion here as well: https://github.com/SAP/openui5/issues.

Since the documentation aspect of your question is addressed now, I will close this issue for now. Thanks again! :-)

@OlMue OlMue closed this as completed Nov 22, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution Valuable Contribution in-discussion This item is being discussed internally
Projects
None yet
Development

No branches or pull requests

5 participants