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

use only lower case for activity and purpose type attributes #56

Open
tcatapano opened this issue May 9, 2017 · 5 comments
Open

use only lower case for activity and purpose type attributes #56

tcatapano opened this issue May 9, 2017 · 5 comments
Labels
Milestone

Comments

@tcatapano
Copy link
Member

Correct type attribute values (purpose and activity) to lower case. This will be assigned to students

@jwayno
Copy link
Contributor

jwayno commented May 9, 2017

I think I've already done this, but if my files (38v-45v) are causing problems, let me know.

@ps2270
Copy link
Contributor

ps2270 commented May 9, 2017

see 39v for other probs--not displaying properly--change in xml

@jwayno
Copy link
Contributor

jwayno commented May 9, 2017

Hmm, I see the problem, but I don't think this is a result of the xml markup. According to Oxygen, the entry is well formed, and I can't find any mistakes. I see Tiana's comment about the .md file, but I don't see how that relates. Ed is pulling from the xml files alone, correct? Also, as far as I can see, the .md file is structured the same way as other .md files, which are displaying properly.

@ps2270
Copy link
Contributor

ps2270 commented May 10, 2017

There were a few oddities in this xml file in GR8975 that I've just fixed--a long space and some misplaced code.

@jwayno
Copy link
Contributor

jwayno commented May 10, 2017

Ah, okay. Thanks for catching those. Unfortunately, I couldn't see them in Oxygen.

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