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

OPHLUDOS-141: Hide language versioning from form for suko #73

Merged
merged 2 commits into from
Oct 11, 2023

Conversation

ChristianVilen
Copy link
Contributor

No description provided.

@ChristianVilen ChristianVilen force-pushed the remove-language-versions-from-suko branch from 62bd53b to 783fc09 Compare October 10, 2023 09:22
@ChristianVilen ChristianVilen force-pushed the remove-language-versions-from-suko branch from 783fc09 to 2fa898d Compare October 10, 2023 09:23
@visadb
Copy link
Contributor

visadb commented Oct 10, 2023

Näissä suko-tehtävissä on SV-sisältöjä. Käydään nää vaik käsin läpi ennen QA:lle viemistä ja siirretään tarvittaessa FI-puolelle kamat.

image

@visadb
Copy link
Contributor

visadb commented Oct 10, 2023

Jossain vaiheessa haluttanee asettaa sisällön kieli automaattisesti käyttöliittymän kieleksi, ja varmaan myös halutaan muistaa käyttäjän valitsema sisältökieli samaan tapaan kuin käyttöliittymän kieli.

Sit pitää muistaa, että suko-tehtävien korteille ja contenteille pitää aina käyttää suomea riippumatta mistään kielivalinnoista. Mahdollinen ongelmaskenaario: käyttäjä käy suko-ohjeissa ja valitseee sisältökieleks ruotsin. Ja menee sit suko-tehtävälistaan, ni jos siellä otetaan sisältökieli samalla mekanismilla kuin muuallakin, ni näytetäänkin tyhjiä sisältöjä sv-kentistä.

Ehdotan, että tehdään Contentin ja Cardin languagelle jo valmiiks tässä pullarissa joku if (sukoassignment) {language=fi}-juttu, ettei kompastuta tähän bugiin sit. Ja se ehkä tekee lukijalle selkeemmäks myös tän outouden, että suko-assignmenteis tallennetaan aina roinat fi-kenttiin.

Copy link
Contributor

@visadb visadb left a comment

Choose a reason for hiding this comment

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

Hyvältä näyttää! Pikkujuttuja vielä

@@ -127,6 +129,7 @@ export async function updateSukoAssignmentForm({

async function fillLdAssignmentTextFields(
page: Page,
exam: Exam,
Copy link
Contributor

Choose a reason for hiding this comment

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

Tän funktion vois nimee fillAssignmentTextFields ku sitä kerran kutsutaan kaikille exameille.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

validettavasti kielitarkenne pullarissa muutetaan ton nimi, joten conflicteja odotellessa. (eli ootetaan et se pr on valmis ja rebasettu tähä)

@ChristianVilen ChristianVilen force-pushed the remove-language-versions-from-suko branch from b860998 to 2fa898d Compare October 11, 2023 10:41
@ChristianVilen ChristianVilen merged commit b755cd8 into main Oct 11, 2023
5 checks passed
@ChristianVilen ChristianVilen deleted the remove-language-versions-from-suko branch October 11, 2023 10:53
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.

2 participants