Merge pull request #40 from starspot/decorators-tweaks

Decorators tweaks

packages/starspot-json-api/src/index.ts

@@ -1,5 +1,5 @@
 export { default as ResourceController, after } from "./resource-controller";
-export { default as Resource, attribute, writable, readOnly, attributes, writableAttributes, hasOne } from "./resource";
+export { default as Resource, attribute, writable, updatable, creatable, readOnly, attributes, writableAttributes, updatableAttributes, creatableAttributes, hasOne } from "./resource";
 export { default as Serializer } from "./serializer";
 export { default as JSONAPI } from "./json-api";
-export { JSONAPIError } from "./exceptions";
+export { JSONAPIError } from "./exceptions";

packages/starspot-json-api/src/resource/decorators.ts

@@ -20,15 +20,21 @@ export function creatable(resource: Resource<any>, attribute: string) {
   fieldFor(resource, attribute, AttributeField).creatable = true;
 }
 
-export function readOnly(resource: Resource<any> | any, attribute?: string | any): any {
+export interface ReadOnlyOptions {
+  ignoreWrites?: boolean;
+}
+
+export function readOnly(resource: Resource<any>, attribute: string): void;
+export function readOnly(options: ReadOnlyOptions): PropertyDecorator;
+export function readOnly(resource: Resource<any> | ReadOnlyOptions, attribute?: string): void | PropertyDecorator {
   if (arguments.length === 1) {
-    let options: any = resource;
+    let options: ReadOnlyOptions = resource;
 
     return function (resource: Resource<any>, attribute: string) {
       let field = fieldFor(resource, attribute, AttributeField);
       field.writable = false;
       field.ignoreUpdateErrors = options && options.ignoreWrites;
-    }
+    };
   } else {
     let field = fieldFor(resource, attribute, AttributeField);
     field.writable = false;
@@ -69,4 +75,4 @@ export function hasOne(relationship: string) {
     let desc = fieldFor(resourceConstructor, relationship, RelationshipField);
     desc.type = "hasOne";
   };
-}
+}