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

New style vancouver-bium-fr #3952

Merged
merged 4 commits into from Apr 21, 2019

Conversation

Projects
None yet
4 participants
@alerac
Copy link
Contributor

commented Feb 25, 2019

  • Added a custom Vancouver style for the
    Lausanne (CH) University Library of Medicine
Alexandre Racine
New style vancouver-bium-fr
* Added a custom Vancouver style for the
  Lausanne (CH) University Library of Medicine
@csl-bot

This comment has been minimized.

Copy link

commented Feb 25, 2019

Awesome! You just created a pull request to the Citation Styles Language styles repository. One of our human volunteers will try to get in touch soon (usually within a week). In the meantime, I will run some automated checks. You should be notified of the results in a few minutes.

If you haven't done so yet, please make sure your style validates and follows all our other Style Requirements.

To update this pull request, visit the "Files changed" tab above, and click on the pencil icon (see below) in the top-right corner of your style to start editing.

If you have any questions, please leave a comment and we'll get back to you. While we usually respond in English, feel free to write in whatever language you're most comfortable.

@csl-bot

This comment has been minimized.

Copy link

commented Feb 25, 2019

😟 There are some issues with your submission. Please check the test report for details.

Alexandre Racine
Corrected errors related to first submission
* Removed unused webpage-details macro and
  corrected url template with http instead
  of https as asked by Travis errors report
@csl-bot

This comment has been minimized.

Copy link

commented Feb 26, 2019

😃 Your submission passed all our automated tests.

@alerac

This comment has been minimized.

Copy link
Contributor Author

commented Mar 6, 2019

Hello,

Do you think there is a chance that our vancouver style for the students of the faculty of Biology and Medicine at the Lausanne University (UNIL) would be accepted into the Zotero repository ?

The CSL style did pass all the automated checks last week but I was wondering if maybe I had to make other corrections. Or if you consider this style would not be eligible ?

Many thanks for all your work.

Alex

@stale

This comment has been minimized.

Copy link

commented Apr 5, 2019

This issue hasn't seen any activity in the past 30 days. It will be automatically closed if no further activity occurs in the next two weeks.

@stale stale bot added the waiting label Apr 5, 2019

@adam3smith

This comment has been minimized.

Copy link
Member

commented Apr 5, 2019

Apologies for the long wait -- will look asap

@stale stale bot removed the waiting label Apr 5, 2019

@alerac

This comment has been minimized.

Copy link
Contributor Author

commented Apr 9, 2019

Apologies for the long wait -- will look asap

No problems. Thank you!

@csl-bot

This comment has been minimized.

Copy link

commented Apr 20, 2019

😃 Your submission passed all our automated tests.

@adam3smith

This comment has been minimized.

Copy link
Member

commented Apr 20, 2019

@rmzelle could you check metadata & merge please? Not sure about naming here.

@csl-bot

This comment has been minimized.

Copy link

commented Apr 21, 2019

😃 Your submission passed all our automated tests.

@rmzelle rmzelle merged commit ff6da14 into citation-style-language:master Apr 21, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rmzelle

This comment has been minimized.

Copy link
Member

commented Apr 21, 2019

Thanks! (@alerac, I changed the title and file name a little to be more in line with the rest of our styles)

@alerac

This comment has been minimized.

Copy link
Contributor Author

commented Apr 24, 2019

Thanks! (@alerac, I changed the title and file name a little to be more in line with the rest of our styles)

Many thanks to you @rmzelle! Ok no problems!

@adam3smith Many thanks also for the clean-up and reformat referenced here

I will just close my fork and propose an update with a new pull request soon. Because in the corrections you made (line 517 removed) the <text variable="note" suffix=" "/> should be present. I managed to have the webpage date (if any) in the Extra field in addition to the website date.

if the line <text variable="note" suffix=" "/> is not present the reference is then not correct. Below an image showing this. Reference 1 is not correct. Reference 2 and 3 are correct.

Many thanks again for your time and all your good work!!

examples

@alerac alerac deleted the alerac:bium branch Apr 24, 2019

@rmzelle

This comment has been minimized.

Copy link
Member

commented Apr 30, 2019

I will just close my fork and propose an update with a new pull request soon.

Note that it's generally not necessary to close/delete a fork on GitHub. You can just follow the instructions at https://github.com/citation-style-language/styles/blob/master/CONTRIBUTING.md#3b-submitting-changes-to-an-existing-style for updating your style.

Because in the corrections you made (line 517 removed) the <text variable="note" suffix=" "/> should be present. I managed to have the webpage date (if any) in the Extra field in addition to the website date.

@adam3smith, do we have a better variable for this? Maybe use the "container" date variable for the website with the expanded syntax, and "issued" for the webpage? (http://docs.citationstyles.org/en/1.0.1/specification.html#date-variables and https://www.zotero.org/support/kb/item_types_and_fields#citing_fields_from_extra)

@adam3smith

This comment has been minimized.

Copy link
Member

commented Apr 30, 2019

Yes, I guess that makes sense, although I still don't quite understand the distinction between a webpage and a website date. What is a website date? Surely it can't be the date a website was first published (because -- who cares?). But if not that, I really have no idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.