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.

Sign up for GitHub

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

Jump to bottom

Added Vedran #437

Merged
merged 1 commit into from
Jul 9, 2015
Merged

Added Vedran #437

merged 1 commit into from
Jul 9, 2015

Conversation

vkaracic
Copy link
Contributor

@vkaracic vkaracic commented Jul 9, 2015

No description provided.

@openedx-webhooks
Copy link

openedx-webhooks commented Jul 9, 2015 •
edited
Loading

Thanks for the pull request, @vkaracic! I've created OSPR-5346 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket:

  • supporting documentation
  • Open edX discussion forum threads
  • timeline information ("this must be merged by XX date", and why that is)
  • partner information ("this is a course on edx.org")
  • any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

@mhoeber
Copy link
Contributor

mhoeber commented Jul 9, 2015

👍

mhoeber added a commit that referenced this pull request Jul 9, 2015
@mhoeber
Merge pull request #437 from edx/vkaracic-authors-addition
45632b8 
Added Vedran
@mhoeber mhoeber merged commit 45632b8 into master Jul 9, 2015
@mhoeber mhoeber deleted the vkaracic-authors-addition branch July 9, 2015 15:01
@openedx-webhooks openedx-webhooks added merged open-source-contribution PR author is not from Axim or 2U labels Jan 8, 2021
@nedbat
Copy link
Contributor

nedbat commented Jan 8, 2021

Hi, sorry for the new comment on this old pull request. We're testing out re-scanning contributions, and our bot added a confusing message that you can ignore. We'll fix that. Sorry about the noise.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
merged open-source-contribution PR author is not from Axim or 2U
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vkaracic @openedx-webhooks @mhoeber @nedbat