Skip to content
This repository was archived by the owner on Sep 8, 2020. It is now read-only.

Commit 285a5fb

Browse files
committed
Merge pull request #184 from angular-ui/dev
Updating v10 candidate to include latest dev changes.
2 parents d38ab68 + 3466d0a commit 285a5fb

File tree

2 files changed

+47
-0
lines changed

2 files changed

+47
-0
lines changed

CONTRIBUTING.md

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,36 @@
1+
## Got a question or problem?
2+
3+
Please, do not open issues for the general support questions as we want to keep GitHub issues for bug reports and feature requests. You've got much better chances of getting your question answered on [StackOverflow](http://stackoverflow.com/questions/tagged/angular-ui-tinymce) where maintainers are looking at questions questions tagged with `angular-ui-tinymce`.
4+
5+
StackOverflow is a much better place to ask questions since:
6+
* there are hundreds of people willing to help on StackOverflow
7+
* questions and answers stay available for public viewing so your question / answer might help someone else
8+
* SO voting system assures that the best answers are prominently visible.
9+
10+
To save your and our time we will be systematically closing all the issues that are requests for general support and redirecting people to StackOverflow.
11+
12+
## You think you've found a bug?
13+
14+
Oh, we are ashamed and want to fix it asap! But before fixing a bug we need to reproduce and confirm it. In order to reproduce bugs we will systematically ask you to provide a _minimal_ reproduce scenario using http://plnkr.co/. Having a live reproduce scenario gives us wealth of important information without going back & forth to you with additional questions like:
15+
* version of AngularJS used
16+
* version of this library that you are using
17+
* 3rd-party libraries used, if any
18+
* and most importantly - a use-case that fails
19+
20+
A minimal reproduce scenario using http://plnkr.co/ allows us to quickly confirm a bug (or point out coding problem) as well as confirm that we are fixing the right problem.
21+
22+
We will be insisting on a minimal reproduce scenario in order to save maintainers time and ultimately be able to fix more bugs. Interestingly, from our experience users often find coding problems themselves while preparing a minimal plunk. We understand that sometimes it might be hard to extract essentials bits of code from a larger code-base but we really need to isolate the problem before we can fix it.
23+
24+
Unfortunately we are not able to investigate / fix bugs without a minimal reproduce scenario using http://plnkr.co/, so if we don't hear back from you we are going to close an issue that don't have enough info to be reproduced.
25+
26+
27+
## You want to contribute some code?
28+
29+
We are always looking for the quality contributions and will be happy to accept your Pull Requests as long as those adhere to some basic rules:
30+
31+
* Please open all pull requests against the `dev` branch.
32+
* Please assure that you are submitting quality code, specifically make sure that:
33+
* You have accompanying tests and all the tests are passing.
34+
* Your PR doesn't break the build; check the Travis-CI build status after opening a PR and push corrective commits if anything goes wrong
35+
* You are using 2 space indentation
36+
* Your commits conform to the conventions established [here](https://github.com/ajoslin/conventional-changelog/blob/master/conventions/angular.md)

README.md

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -69,6 +69,10 @@ If you add the ng-model directive to same the element as ui-tinymce then the tex
6969

7070
_The ui-tinymce directive stores the configuration options as specified in the [TinyMCE documentation](http://www.tinymce.com/wiki.php/Configuration) and expects the model value to be a html string or raw text, depending on whether `raw` is `true` (default value is `false`)._
7171

72+
**Note:** Make sure you using scopes correctly by following [this wiki page](https://github.com/angular/angular.js/wiki/Understanding-Scopes). If you are having issues with your model not updating, make sure you have a '.' in your model.
73+
74+
> This issue with primitives can be easily avoided by following the "best practice" of always have a '.' in your ng-models – watch 3 minutes worth. Misko demonstrates the primitive binding issue with ng-switch.
75+
7276
## Options
7377

7478
The directive supports all of the standard TinyMCE initialization options as listed [here](http://www.tinymce.com/wiki.php/Configuration).
@@ -100,3 +104,10 @@ myAppModule.controller('MyController', function($scope) {
100104
<textarea ui-tinymce="tinymceOptions" ng-model="tinymceModel"></textarea>
101105
</form>
102106
```
107+
108+
----
109+
110+
111+
# Contributing to the project
112+
113+
We are always looking for the quality contributions! Please check the [CONTRIBUTING.md](CONTRIBUTING.md) for the contribution guidelines.

0 commit comments

Comments
 (0)