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

Remove mrn_option_field when uninstall plugin #3

Open
Nann opened this issue Oct 4, 2017 · 3 comments
Open

Remove mrn_option_field when uninstall plugin #3

Nann opened this issue Oct 4, 2017 · 3 comments

Comments

@Nann
Copy link
Contributor

Nann commented Oct 4, 2017

ผมลองตรวจสอบดูแล้วครับ มีข้อมูลค้างในฐานข้อมูล เมื่อเราถอนการติดตั้งปลั๊กอิน mrn_option_field ใน _options ครับ

@jojoee
Copy link
Member

jojoee commented Oct 4, 2017

ตรงนี้เพราะ ผมไม่ได้เอาออกครับ
อันนี้ผมลองคิดไปรอบนึงเหมือนกัน อยากจะมาปรึกษาครับ

ที่ผมคิดคือ user ลบ plugin ไป แล้วโหลดกลับมาใหม่
user จะสามารถใช้งานได้เลย (คือใช้ค่า setting เดิมที่เคยตั้งค่าไว้)

@Nann
Copy link
Contributor Author

Nann commented Oct 4, 2017

ผมคิดแบบนี้ครับ ปลั๊กอินตัวนี้มีการตั้งค่าอยู่ 2 ส่วน คือ Grayscale กับ Black Ribbon ตัวปลั๊กอินเองไม่ได้มีความซับซ้อนอะไรมาก หน้าปลั๊กอินก็มี link สำหรับคลิกเพื่อมายัง หน้าการตั้งค่าของปลั๊กอิน User สามารถปรับการตั้งค่าใหม่ได้

คิดว่าถ้าหากถอนการติดตั้งไปแล้ว อยากให้ลบข้อมูลออกไปเลยครับ อีกอย่าง ตอนนี้ยังไม่มี Feedback ผมเลยอยากศึกษาหาความรู้เพิ่มเติมเองด้วยครับ

ส่วนนี้ให้ผมทำแล้ว Pull Request มาดูก่อนว่าผมเขียนถูกวิธีหรือไม่ดีไหมครับ แล้วคุณโจ้ค่อยตัดสินใจว่าจะ Merge เข้า Master หรือไม่ ค่อยเป็นอีกเรื่องดีไหมครับ

เพราะผมค่อนข้างชัดเจนว่า อยากหาความรู้เพิ่มเติม ^^

@jojoee
Copy link
Member

jojoee commented Oct 4, 2017

ok ครับ รอ PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants