Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Updating enums Script for Displaying Languages #268

Merged
merged 4 commits into from
Jan 6, 2025

Conversation

breezy-codes
Copy link

@breezy-codes breezy-codes commented Nov 19, 2024

Description

This pull request updates the components.cjs file to support a new feature I've introduced in the translator. Previously, enum signatures were displayed only for C++ on the SplashKit website. The changes in the translator now allow for pulling and displaying enum syntax for multiple languages, enhancing the website's documentation capabilities.

To demonstrate the functionality and ensure compatibility, the updated api.json is included in this pull request. It is essential for being able to preview the changes, as the updates introduce modifications that are incompatible with the previous build configuration.

The pull request for the translator change is here - thoth-tech/splashkit-translator#17

Type of change

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

Testing was conducted using the updated api.json generated from the modified translator. This ensured that the new enum formatting was accurately reflected in the documentation and did not introduce errors during the build process.

Testing Checklist

  • Tested in latest Chrome
  • Tested in latest Firefox
  • npm run build
  • npm run preview

Copy link

netlify bot commented Nov 19, 2024

Deploy Preview for splashkit-io ready!

Name Link
🔨 Latest commit 6ed7d64
🔍 Latest deploy log https://app.netlify.com/sites/splashkit-io/deploys/677b6f3f6cb4ed0008c6015d
😎 Deploy Preview https://deploy-preview-268--splashkit-io.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

@EthanRin EthanRin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a great update for the SplashKit website that enhances the clarity and capabilities of the documentation page. I have pulled this PR and tested on my local fork and have found no error. This code is well structured with comments on hard to understand area, enhancing code readability for further students.

@omckeon omckeon self-assigned this Jan 6, 2025
@omckeon omckeon merged commit 266133f into thoth-tech:main Jan 6, 2025
4 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants