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

Add generator tag #60

Merged
merged 1 commit into from
Jun 28, 2020
Merged

Add generator tag #60

merged 1 commit into from
Jun 28, 2020

Conversation

saerdnaer
Copy link
Member

@saerdnaer saerdnaer commented Dec 25, 2019

Copy link
Contributor

@meise meise left a comment

Choose a reason for hiding this comment

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

Wenn das auch das frab macht spricht aus eigentlich nix dagegen, das auch optional zuzulassen.

@meise
Copy link
Contributor

meise commented Dec 25, 2019

Darf natürlich nicht gemerged werden solange die Tests failen.

@johnjohndoe
Copy link
Collaborator

Könnt ihr bitte ein Beispiel XML (vorher/nachher) posten, damit ich einschätzen kann, ob der Parser für die 36C3 Fahrplan App damit umgehen kann?

@rixx
Copy link

rixx commented Dec 26, 2019

Wenn das auch das frab macht spricht aus eigentlich nix dagegen, das auch optional zuzulassen.

Das macht nicht "auch" das frab, das macht das frab seit Januar, und validiert deshalb seither nicht mehr gegen eure XSD. pretalx musste sich jetzt entscheiden, ob es mit frab oder mit der schedule.xsd kompatibel ist, und hat sich fürs frab entschieden.

Könnt ihr bitte ein Beispiel XML (vorher/nachher) posten, damit ich einschätzen kann, ob der Parser für die 36C3 Fahrplan App damit umgehen kann?

Wenn die Fahrplan-App mit aktuellem frab-Output umgehen kann, scheint diese Änderung ja kein Problem zu machen.

@johnjohndoe
Copy link
Collaborator

Wenn die Fahrplan-App mit aktuellem frab-Output umgehen kann, scheint diese Änderung ja kein Problem zu machen.

Ich hoffe, das ist/bleibt so. Dennoch würde gerne verstehen, wo das generator Attribut auftaucht? Ich konnte es bisher nicht finden:

Kann jemand bitte meine 👃 auf die richtige Stelle stoßen?

@rixx
Copy link

rixx commented Dec 26, 2019

Hier liegt der Tag vor.

@saerdnaer
Copy link
Member Author

@johnjohndoe Dieser PR ist nur für das XSD, in den schedule merging scripte baue ich es erst ein wenn du dein okay gibst :-)

@johnjohndoe
Copy link
Collaborator

@rixx Danke. @saerdnaer Okay.

@rixx
Copy link

rixx commented Apr 19, 2020

Ping: ich fänds schön, wenn die schedule.xsd wieder mit frab und pretalx kompatible wäre, weils mir das Leben leichter macht, wenn ich nach der xsd statt dem frab-Repo coden kann.

@saerdnaer saerdnaer merged commit b986ac3 into master Jun 28, 2020
@saerdnaer saerdnaer deleted the add-generator-tag branch June 28, 2020 20:00
@johnjohndoe
Copy link
Collaborator

Gibt es schon irgendwo eine schedule.xml Beispielgenerierung, mit <generator> Element? Wird diese rückwirkend auf den 36C3-Schedule angewendet?

@rixx
Copy link

rixx commented Jun 30, 2020

Sowohl pretalx als auch frab generieren seit geraumer Zeit schedule.xml mit generator. Hier zB.

@a-tze
Copy link
Contributor

a-tze commented Jun 30, 2020

Gibt es schon irgendwo eine schedule.xml Beispielgenerierung, mit <generator> Element? Wird diese rückwirkend auf den 36C3-Schedule angewendet?

Da ist der Tag eigentlich drin: https://fahrplan.events.ccc.de/congress/2019/Fahrplan/schedule.xml

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.

5 participants