Skip to content

Issue template#1710

Merged
taylortom merged 4 commits intoadaptlearning:developfrom
CanStudios:Issue-template
Aug 26, 2017
Merged

Issue template#1710
taylortom merged 4 commits intoadaptlearning:developfrom
CanStudios:Issue-template

Conversation

@canstudios-louisem
Copy link
Copy Markdown
Contributor

@canstudios-louisem canstudios-louisem commented Aug 18, 2017

This will hopefully give people some guidance on what should go into an in issue, both bugs and feature requests.

It would be great to get this in as part of 0.4.0 as it will be merged with master soonish but I appreciate its a bit last minute.

Docs on issue templates here.

Issue: #1709

Copy link
Copy Markdown
Contributor

@canstudios-nicolaw canstudios-nicolaw left a comment

Choose a reason for hiding this comment

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

I think this would be useful. Steps to reproduce would certainly help when it comes to testing.

Copy link
Copy Markdown
Member

@taylortom taylortom left a comment

Choose a reason for hiding this comment

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

Looks good to me, happy to approve with the following addition:

Comment thread .github/ISSUE_TEMPLATE.md
## Versions
- Authoring Tool Version:
- Framework Version:
- Operating System:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can we also have a section to specify the browsers affected?

Comment thread .github/ISSUE_TEMPLATE.md Outdated
@@ -0,0 +1,28 @@
# BUG Template

## Expected Behavior
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Behaviour

Comment thread .github/ISSUE_TEMPLATE.md Outdated

## Expected Behavior

## Actual Behavior
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Behaviour

Comment thread .github/ISSUE_TEMPLATE.md Outdated

## Use Case

## Current Work Around
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Workaround

@taylortom taylortom merged commit b63a403 into adaptlearning:develop Aug 26, 2017
@canstudios-louisem canstudios-louisem deleted the Issue-template branch October 18, 2017 09:43
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.

4 participants