-
Notifications
You must be signed in to change notification settings - Fork 278
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
#288 Add Marking Scales for non-USA Courses #328
#288 Add Marking Scales for non-USA Courses #328
Conversation
Can we merge this one? I remember that there was consensus that |
@evanplaice @thomasdavis what do you think? |
@dandydev I think we can keep it as I think the reason was
Although it seems American, it is not meant to be. |
The quote says that |
Yeah, I prefer |
This one cannot be merged automatically, I will resolve it manually and merge. Probably tonight (GMT+2) |
I do like score and a min, max, that can easily be shown as a percentage. That covers all min, max scales and the type being a free form covers explanation.
|
@stp-ip Makes a lot of sense. Though over the five years it is quite clear that theme developers really value simplicity, and also a lot of people have been happy to keep their resume rather elementary. |
Great thing about this is that score and type are enough so it can default back to simple. |
I still think it's too complicated for the majority of use cases. In fact, I'd recommend that we call it |
Not sure I fully agree. Giving a framework with flexibility on what is being used > making free form fields do everything. |
I'm in favor of being specific and rigorous in places where the impact/significance of that part of a resume is big. But in all my years of hiring people internationally, I have rarely seen people specify their educational grades in a way more specific than just one value (be it a percentage, number). And to be honest, I've rarely seen people actually specify grades/scores at all. And don't get me wrong, I acknowledge that being able to specify a grade is useful, and I also acknowledge that there is a variety of ways in which people might want to do this. But I strongly feel that we should not try to make a complex specification that tries to capture the myriad of ways in which somebody might want to specify their grade. That's why I want to give people all the options. To be honest, personally, I don't want to have to specify |
Thanks for giving the context. Personally had various requests or feedback especially internationally, where a grade format wasn't widely known and hiring internationally was easier with a percentage number or at least have an ability to visually say great, good, not good etc. The great thing about the schema being in the background is that not everything has to be exposed, but having it set even just to set a precedent would be good in my view. Only setting value and assuming a context of it being GPA is fine, so is adding an additional type or for other type of scenarios a range. The specific idea to use score instead of grade was the re-usability of the scores for various parts of the schema. It makes it easier for theme developers to reuse design and tooling. |
I'm not hung-up on the term With regards to the JSON Schema type of {
"score": 5.3
} they have to fill in this:
And that would only be so we can support what feels to me as edge cases. I don't think @stp-ip and me are going to come to a consensus there, so I propose @thomasdavis makes a decision and we both live with it :) |
I think we sort of agree on a few things so. For example I agree that score and/or type are the fields that are most important. Difference is I think the schema should be a bit wider than the normal use case and let the ecosystem tooling handle the easy to use cases. Aka the schema could accommodate the full score object, but 90% of the editors might only expose "score" the field and that is fine, but having a rule set within the schema for the other 10% makes the overall usefulness of the schema bigger in my view. Totally agree, that we can't do this for everything, but especially with the reusable nature of score, it might still make sense. Happy to hear Thomas's thoughts :) Sidenote |
+1 in my experience in hiring, the grade has always been easily conveyed or non important. More often than not we will stick with strings over flexibility, the only thing holding the ecosystem together is the simpleness. If people absolutely require more complex fields, they are free to do so, by a) forking the theme they want to use and updating it and b) adding their fields as they wish to their resume.json. By virture of the entire project being open source, they could fork everything. Until a mass of people have pitch forks burning down the README I think we can err on the side of basic. As for
|
ok, so for now: Regarding the name, I'm actually starting to lean more towards If we can come to an agreement on the name of the field, I will adjust the PR, resolve conflicts and merge. |
Would love to get a maintainer call done before merging, but definitely not opposed :) In my non native english speaker mind grade is more tailored towards school compared to a score, that could be for a certificate, training or the like. |
But if we want to choose one for |
score |
Test on non-EOL LTS versions of Node
Node <10 is officially end-of-life. 10+ will enable built-in promises as well as many other useful features.
Test every aspect of the validator
Node 10+ no longer requires a polyfill to use use promises.
* Specify the Correct Data Type - basics.network.url -> uri - publications.url -> uri - meta.canonical -> uri * Fix Mistake
re: ["remove mention of migration.js since that doesn't exist"](#349) If any of the removed content here is still meaningful, we could add it to a roadmap.
The JSONSchema date format is too strict (ie YYYY-MM-DD). This new validation pattern is still ISO8601 compliant but allows for - YYYY-MM-DD - YYYY-MM - YYYY
Now that CI is covered by GH Actions, Travis is redundant. * Remove .travis.yml * Fix README.md Badges
Migrate the change log to CHANGELOG.md and back-fill missing data.
I tried to fix the merge conflict, but it pulled in a lot of commits. |
I've created a new PR with the same change but also with passing tests and no merge conflicts. Have a look at #391 |
Fixed by #391 |
#288