Skip to content

The first version of translation functionality in Vue templates, whe…#1662

Closed
miguelvaara wants to merge 1 commit intoissue1527-translation-symfonyfrom
issue1523translate-messages-shown-by-vue-components
Closed

The first version of translation functionality in Vue templates, whe…#1662
miguelvaara wants to merge 1 commit intoissue1527-translation-symfonyfrom
issue1523translate-messages-shown-by-vue-components

Conversation

@miguelvaara
Copy link
Contributor

…re translations are visible in the browser and the translation is read at the correct point in the code

Reasons for creating this PR

Link to relevant issue(s), if any

  • Closes #

Description of the changes in this PR

Known problems or uncertainties in this PR

Checklist

  • phpUnit tests pass locally with my changes
  • I have added tests that show that the new code works, or tests are not relevant for this PR (e.g. only HTML/CSS changes)
  • The PR doesn't reduce accessibility of the front-end code (e.g. tab focus, scaling to different resolutions, use of .sr-only class, color contrast)
  • The PR doesn't introduce unintended code changes (e.g. empty lines or useless reindentation)

…re translations are visible in the browser and the translation is read at the correct point in the code
@sonarqubecloud
Copy link

@miguelvaara miguelvaara self-assigned this Aug 21, 2024
@miguelvaara miguelvaara added this to the 3.0 milestone Aug 21, 2024
@osma osma changed the title The first version of translation functionality in Twig templates, whe… The first version of translation functionality in Vue templates, whe… Aug 22, 2024
@miguelvaara
Copy link
Contributor Author

The next step is to prepare this Draft PR for a design review.

It is needed to briefly list what is intended to be demonstrated in the design review and where the focus of the future developement should be directed.

For example, listing languages in the frontend code is unnecessary, as we can directly refer to Skosmos' runtime language, and there are also unnecessary empty spaces in the code ... among other things .. etc.

@osma
Copy link
Member

osma commented Oct 9, 2024

Should this PR be closed? I think it's been superseded by #1690

@osma osma deleted the branch issue1527-translation-symfony October 10, 2024 13:22
@osma osma closed this Oct 10, 2024
@osma osma modified the milestones: 3.x, 3.0 Jan 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants