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

本番用にminify化するとgetModelNameが意図したとおりの結果にならなくなる #43

Open
kahirokunn opened this issue Dec 29, 2018 · 0 comments

Comments

@kahirokunn
Copy link
Contributor

getModelNameでconstructor名からmodel名を構築しているコードが見受けられましたが、constructor名がminify化すると変わってしまいます。

現在は以下のような手段でその影響から逃れました

import * as Pring from 'pring'

const modelName = 'user'

export default class User extends Pring.Base {
  public static getModelName() {
    return modelName
  }
  public getModelName() {
    return modelName
  }
}

constructorから生成するのはとても良い手法の1つなのですが、自分でmodel名を指定するようにするのも良いのかなとは思いますが、static変数とインスタンス変数両方に定義するのもイケてないなぁとは思っています。
何か良いアイディアでこの辺のDXを上げられないかここで出し合って議論したいと思い、issueを作らせて頂きました。

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

No branches or pull requests

1 participant