+
Skip to content

Conversation

Armored-Dragon
Copy link
Contributor

@Armored-Dragon Armored-Dragon commented Jul 1, 2025

This PR provides an updated metadata generator that provides a form that will generate metadata for both the current more app, and the PR More app overte-org/overte#1615.

This makes some notable changes to the repo:

  • The generator webpage is now located at 'metadata_generator' as opposed to metadata_js_generator.
    • The page still exists but it serves as a redirect to metadata_generator.
    • This was made because we are no longer generating a Javascript function with the PR More app.
  • You can now provide your own repository and have the generator work based on that.
  • Index.html was also touched up as on my monitors the text seem needlessly small. Some language was also changed and moved around for brevity.

TODO:

  • Test output for consistent results
  • Ensure output does not break the existing More application

Before this is merged I would also like to see if using SCSS is fine when it comes to contributing to the master branch. SCSS is, to oversimplify, a preprocessor for CSS. I don't think it is a major overhead in the long run but I would figure I get this discussion settled now opposed to several months down the line.

@Armored-Dragon Armored-Dragon self-assigned this Jul 1, 2025
@Armored-Dragon Armored-Dragon added documentation Improvements or additions to documentation enhancement New feature or request question Further information is requested needs CR needs QA labels Jul 1, 2025
@Armored-Dragon Armored-Dragon marked this pull request as ready for review July 14, 2025 19:32
@AleziaKurdis
Copy link
Contributor

Retaken under PR #113

We can close this PR #107

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request needs CR needs QA question Further information is requested To close

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载