Re: On adding comments to database schema

2021-05-08 Thread Jared Chung
Thank you Tom. I'll go ahead and do that. I may seek some advice 
in django-core-mentorship since this is my first time working on a patch.
KimSoungRyoul I hope you don't mind that I proceed with reassignment. If 
you remain interested, please let me know and I'd love to collaborate with 
you on it together. Otherwise I'll give it my best shot. 

On Thursday, May 6, 2021 at 6:39:45 AM UTC-7 t...@carrick.eu wrote:

> It's funny that you bring this up, I was actually looking at this today 
> and thinking about picking it up next week.
>
> Jared, the PR was closed without any comment from the author and is 
> seemingly abandoned. I think it's safe for you to assign the ticket to 
> yourself and continue the work in a new PR at this point - that was going 
> to be my plan, anyway.
>
> Tom
>
> On Thu, 6 May 2021 at 15:16, Jared Chung  wrote:
>
>> KimSoungRyoul would you like help with writing the docs and tests for 
>> this? I'd like to contribute to Django in some way and this is a feature my 
>> company would greatly value (we have Data Scientists and DBAs interacting 
>> with db tables managed by Django, so having comment documentation at the DB 
>> level would have a huge positive impact). It looks like you received 
>> feedback on your PR  and 
>> Mariusz pointed out that docs and tests are needed. I'd be happy to help 
>> out with the docs and tests if I can. You're marked as ticket owner right 
>> now on ticket 18468 . 
>> Please let me know!
>> *(Disclosure: I'm new to contributing to Django, and hopeful that this 
>> can be a good ticket for me to start with. I've been using Django now for a 
>> decade and am excited to start giving back!)*
>>
>> On Saturday, May 2, 2020 at 10:24:46 PM UTC-7 kimsou...@gmail.com wrote:
>>
>>> I'm sorry if I sounded like I was pushing you. 
>>>
>>> even missing checklist
>>>
>>>
>>> 2020년 5월 3일 일요일 오전 1시 42분 2초 UTC+9, Mariusz Felisiak 님의 말:
>>>
 Asking for a review on multiple channels doesn't help. You need to be 
 patient, we have many PRs in review queue. Please check "Patch review 
 checklist" [1]. At first glance, docs are missing.

 [1] 
 https://docs.djangoproject.com/en/3.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist

>>> -- 
>>
> You received this message because you are subscribed to the Google Groups 
>> "Django developers (Contributions to Django itself)" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to django-develop...@googlegroups.com.
>>
> To view this discussion on the web visit 
>> https://groups.google.com/d/msgid/django-developers/62705a74-f16d-42c3-b440-4e3e7628351cn%40googlegroups.com
>>  
>> 
>> .
>>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/6a477cf1-7cfa-4fd3-97f2-b2a9d62b2e1fn%40googlegroups.com.


Re: On adding comments to database schema

2021-05-06 Thread Tom Carrick
It's funny that you bring this up, I was actually looking at this today and
thinking about picking it up next week.

Jared, the PR was closed without any comment from the author and is
seemingly abandoned. I think it's safe for you to assign the ticket to
yourself and continue the work in a new PR at this point - that was going
to be my plan, anyway.

Tom

On Thu, 6 May 2021 at 15:16, Jared Chung  wrote:

> KimSoungRyoul would you like help with writing the docs and tests for
> this? I'd like to contribute to Django in some way and this is a feature my
> company would greatly value (we have Data Scientists and DBAs interacting
> with db tables managed by Django, so having comment documentation at the DB
> level would have a huge positive impact). It looks like you received
> feedback on your PR  and
> Mariusz pointed out that docs and tests are needed. I'd be happy to help
> out with the docs and tests if I can. You're marked as ticket owner right
> now on ticket 18468 . Please
> let me know!
> *(Disclosure: I'm new to contributing to Django, and hopeful that this can
> be a good ticket for me to start with. I've been using Django now for a
> decade and am excited to start giving back!)*
>
> On Saturday, May 2, 2020 at 10:24:46 PM UTC-7 kimsou...@gmail.com wrote:
>
>> I'm sorry if I sounded like I was pushing you.
>>
>> even missing checklist
>>
>>
>> 2020년 5월 3일 일요일 오전 1시 42분 2초 UTC+9, Mariusz Felisiak 님의 말:
>>
>>> Asking for a review on multiple channels doesn't help. You need to be
>>> patient, we have many PRs in review queue. Please check "Patch review
>>> checklist" [1]. At first glance, docs are missing.
>>>
>>> [1]
>>> https://docs.djangoproject.com/en/3.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist
>>>
>> --
> You received this message because you are subscribed to the Google Groups
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to django-developers+unsubscr...@googlegroups.com.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/django-developers/62705a74-f16d-42c3-b440-4e3e7628351cn%40googlegroups.com
> 
> .
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/CAHoz%3DMbaEAaV1nt9dL576RvLzDSL_LacJ-fRLAe4Bwo416HDfA%40mail.gmail.com.


Re: On adding comments to database schema

2021-05-06 Thread Jared Chung
KimSoungRyoul would you like help with writing the docs and tests for this? 
I'd like to contribute to Django in some way and this is a feature my 
company would greatly value (we have Data Scientists and DBAs interacting 
with db tables managed by Django, so having comment documentation at the DB 
level would have a huge positive impact). It looks like you received 
feedback on your PR  and 
Mariusz pointed out that docs and tests are needed. I'd be happy to help 
out with the docs and tests if I can. You're marked as ticket owner right 
now on ticket 18468 . Please 
let me know!
*(Disclosure: I'm new to contributing to Django, and hopeful that this can 
be a good ticket for me to start with. I've been using Django now for a 
decade and am excited to start giving back!)*

On Saturday, May 2, 2020 at 10:24:46 PM UTC-7 kimsou...@gmail.com wrote:

> I'm sorry if I sounded like I was pushing you. 
>
> even missing checklist
>
>
> 2020년 5월 3일 일요일 오전 1시 42분 2초 UTC+9, Mariusz Felisiak 님의 말:
>
>> Asking for a review on multiple channels doesn't help. You need to be 
>> patient, we have many PRs in review queue. Please check "Patch review 
>> checklist" [1]. At first glance, docs are missing.
>>
>> [1] 
>> https://docs.djangoproject.com/en/3.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist
>>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/62705a74-f16d-42c3-b440-4e3e7628351cn%40googlegroups.com.


Re: On adding comments to database schema

2021-05-06 Thread Jared Chung
KimSoungRyoul it looks like there is feedback on the PR you created (PR 
#12605 ) and Marius also 
pointed out that docs and tests are missing. Would you like help with the 
docs and tests? We've recently added some data scientists and dbas to our 
team and I immediately saw the significant value in ticket 28822 
. You're marked as the ticket 
owner right now, but I'd love to help out with docs and tests if you feel 
that the rest of the implementation is solid (or if you are willing to 
update your PR based on the feedback from Mads Jensen (atombrella).
*(Full disclosure: I'm new to contributing to django, but hoping that this 
can be my first contribution. I've been thankful to use Django at work for 
9 years now!)* 


On Saturday, May 2, 2020 at 10:24:46 PM UTC-7 kimsou...@gmail.com wrote:

> I'm sorry if I sounded like I was pushing you. 
>
> even missing checklist
>
>
> 2020년 5월 3일 일요일 오전 1시 42분 2초 UTC+9, Mariusz Felisiak 님의 말:
>
>> Asking for a review on multiple channels doesn't help. You need to be 
>> patient, we have many PRs in review queue. Please check "Patch review 
>> checklist" [1]. At first glance, docs are missing.
>>
>> [1] 
>> https://docs.djangoproject.com/en/3.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist
>>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/0f963398-7733-4cd6-9b5f-600945030b36n%40googlegroups.com.


Re: On adding comments to database schema

2020-05-02 Thread SoungRyoul Kim
I'm sorry if I sounded like I was pushing you. 

even missing checklist


2020년 5월 3일 일요일 오전 1시 42분 2초 UTC+9, Mariusz Felisiak 님의 말:
>
> Asking for a review on multiple channels doesn't help. You need to be 
> patient, we have many PRs in review queue. Please check "Patch review 
> checklist" [1]. At first glance, docs are missing.
>
> [1] 
> https://docs.djangoproject.com/en/3.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/d50ae4c2-86d8-4071-88ac-cde6b3a364a6%40googlegroups.com.


Re: On adding comments to database schema

2020-05-02 Thread Mariusz Felisiak
Asking for a review on multiple channels doesn't help. You need to be 
patient, we have many PRs in review queue. Please check "Patch review 
checklist" [1]. At first glance, docs are missing.

[1] 
https://docs.djangoproject.com/en/3.0/internals/contributing/writing-code/submitting-patches/#patch-review-checklist

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/f756bda6-c044-47c0-9317-7949eb114398%40googlegroups.com.


Re: On adding comments to database schema

2020-05-02 Thread SoungRyoul Kim


Hi felixxm

I want this feature to be reflected in Django.3.1

I know this feature that many people won't use.

but some people who communicate with DBA would be helpful this feature

sometimes ORM take control from DBA


*like these situation...*

DBAs: we want to know comment(help_text) not to *.py  could you use COMMENT SQL 
in migrationFile?

Developers: No, It's Impossible because django ORM does not support COMMENT SQL

DBAs:  that means .. should we refer *.py file everytimes even though 
COMMENT SQL exists but can't use?  
DBAs: it's illogical

Developers: but if we use COMMENT SQL, we customize everytimes to generate 
migrate files
Developers: it's inefficient


That's one of the reasons for avoiding ORM.


If you don't mind me asking...
could you give me a feedback to this feature?

https://github.com/django/django/pull/12605

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/29a5b1c4-21e1-4784-8c00-40778d074ea5%40googlegroups.com.


Re: On adding comments to database schema

2020-04-10 Thread SoungRyoul Kim


add parameter "db_column_comment" to Field() 

and. db_table_comment

class AModel(models.Model):
 aaa = model.CharField(help_text="i am help_text", db_column_comment="i am 
db_comment",~~~)
 
 class Meta:
   db_table = "a_model_example_name"
   db_table_comment ="this is a_model comment "


-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/b7b639bd-5f18-4492-95a5-5bf6cbe89f02%40googlegroups.com.


Re: On adding comments to database schema

2020-03-23 Thread SoungRyoul Kim


i assign it to me
https://code.djangoproject.com/ticket/18468 


i will close this pullRequest caused by imperfection
​https://github.com/django/django/pull/12605

I will bring a new PullRequest as soon as possible.

in at least a week

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/8f343f6f-3ecb-4c3e-b8f6-7c9018480505%40googlegroups.com.


Re: On adding comments to database schema

2019-06-05 Thread Adam Johnson
Mauricio, the ticket tracker doesn't tend to get out of sync, so you can
see the ticket is still open and waiting for an implementation:
https://code.djangoproject.com/ticket/18468

If you want to help, you can start with the "writing your first patch for
django" tutorial: https://docs.djangoproject.com/en/dev/intro/contributing/

On Wed, 5 Jun 2019 at 06:22, vanadium23  wrote:

> I've made it as a separate django libm but only for PostgresSQL:
> https://github.com/vanadium23/django-db-comments
>
> среда, 5 июня 2019 г., 0:29:21 UTC+3 пользователь Mauricio Cordero написал:
>>
>> Hello all, just wondering if this feature was ever implemented.
>>
> --
> You received this message because you are subscribed to the Google Groups
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to django-developers+unsubscr...@googlegroups.com.
> To post to this group, send email to django-developers@googlegroups.com.
> Visit this group at https://groups.google.com/group/django-developers.
> To view this discussion on the web visit
> https://groups.google.com/d/msgid/django-developers/a8bf7145-d878-4ef0-b39c-3ea3ad7f62ac%40googlegroups.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>


-- 
Adam

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/CAMyDDM3tsvDezOToRWiA1vh4c6bkHyhsVyU2g-QOtP5CaS%3D%3DSw%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.


Re: On adding comments to database schema

2019-06-04 Thread vanadium23
I've made it as a separate django libm but only for PostgresSQL:
https://github.com/vanadium23/django-db-comments

среда, 5 июня 2019 г., 0:29:21 UTC+3 пользователь Mauricio Cordero написал:
>
> Hello all, just wondering if this feature was ever implemented. 
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/a8bf7145-d878-4ef0-b39c-3ea3ad7f62ac%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: On adding comments to database schema

2019-06-04 Thread Mauricio Cordero
Hello all, just wondering if this feature was ever implemented. 

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/cda2cd66-8076-47ff-9462-1a1a1292efc5%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: On adding comments to database schema

2018-03-18 Thread Aymeric Augustin
Hello,

My gut feeling is that I'd rather implement this in the migrations engine (your 
option 1) than bolt it on (the option 2).

Disclaimer — I don't know the migrations engine all that well.

Best regards,

-- 
Aymeric.



> On 18 Mar 2018, at 07:12, vanadium23  wrote:
> 
> Hi, community.
> 
> I've been stuck at realization, because Postgres and oracle have a syntax 
> like comment on {table}.{column} for storing comments, so this needs to be 
> done after table/column creation.
> 
> So there are two ways:
> 1. Add it to post migrate signal, as for content types. But I can implement 
> it as a third-party lib
> 2. Add this SQL after database creation in schema.py
> Which way is better?
> 
> воскресенье, 11 марта 2018 г., 11:34:46 UTC+3 пользователь vanadium23 написал:
> That's good. I'll prepare the patch for review.
> 
> суббота, 10 марта 2018 г., 10:27:30 UTC+3 пользователь Aymeric Augustin 
> написал:
> I think we could implement that feature.
> 
> (Yes I changed my mind from six years ago when I wontfix'd the ticket.)
> 
> -- 
> Aymeric.
> 
> 
> 
>> On 8 Mar 2018, at 11:00, Jani Tiainen > wrote:
>> 
>> Oracle supports comments as well.
>> 
>> 7.3.2018 2.52 ip. "vanadium23" > kirjoitti:
>> Hello, fellows.
>> 
>> There was once a proposal about ability to add comments to table/columns in 
>> postgres: https://code.djangoproject.com/ticket/18468 
>> 
>> I re-read discussion in ticket, and it has ended with another proposal, that 
>> now migrations is within Django, so we can add this feature.
>> 
>> I want to make it as a third-party libs, but has come to thought that need 
>> to extend SchemaEditor, but it's not good for maintainability.
>> Also, not only postgres has this feature, but also MySQL.
>> 
>> Motivation for this feature is that some sort of users can view code 
>> comments without access to VCS of the project.
>> 
>> -- 
>> You received this message because you are subscribed to the Google Groups 
>> "Django developers (Contributions to Django itself)" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to django-develop...@googlegroups.com <>.
>> To post to this group, send email to django-d...@googlegroups.com <>.
>> Visit this group at https://groups.google.com/group/django-developers 
>> .
>> To view this discussion on the web visit 
>> https://groups.google.com/d/msgid/django-developers/484264c0-b7d1-4264-b8b5-8857847f6b53%40googlegroups.com
>>  
>> .
>> For more options, visit https://groups.google.com/d/optout 
>> .
>> 
>> -- 
>> You received this message because you are subscribed to the Google Groups 
>> "Django developers (Contributions to Django itself)" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to django-develop...@googlegroups.com <>.
>> To post to this group, send email to django-d...@googlegroups.com <>.
>> Visit this group at https://groups.google.com/group/django-developers 
>> .
>> To view this discussion on the web visit 
>> https://groups.google.com/d/msgid/django-developers/CAHn91oesnBvY-mWXoHWa3sf8Vri%2B1qwRHat1yF%2B8Sdm0pU-LhQ%40mail.gmail.com
>>  
>> .
>> For more options, visit https://groups.google.com/d/optout 
>> .
> 
> 
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to django-developers+unsubscr...@googlegroups.com 
> .
> To post to this group, send email to django-developers@googlegroups.com 
> .
> Visit this group at https://groups.google.com/group/django-developers 
> .
> To view this discussion on the web visit 
> https://groups.google.com/d/msgid/django-developers/9ff6c9f5-045e-4bc7-8331-060471b012e2%40googlegroups.com
>  
> .
> For more options, visit https://groups.google.com/d/optout 
> .

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from 

Re: On adding comments to database schema

2018-03-18 Thread vanadium23
Hi, community.

I've been stuck at realization, because Postgres and oracle have a syntax 
like comment on {table}.{column} for storing comments, so this needs to be 
done after table/column creation.

So there are two ways:
1. Add it to post migrate signal, as for content types. But I can implement 
it as a third-party lib
2. Add this SQL after database creation in schema.py
Which way is better?

воскресенье, 11 марта 2018 г., 11:34:46 UTC+3 пользователь vanadium23 
написал:
>
> That's good. I'll prepare the patch for review.
>
> суббота, 10 марта 2018 г., 10:27:30 UTC+3 пользователь Aymeric Augustin 
> написал:
>>
>> I think we could implement that feature.
>>
>> (Yes I changed my mind from six years ago when I wontfix'd the ticket.)
>>
>> -- 
>> Aymeric.
>>
>>
>>
>> On 8 Mar 2018, at 11:00, Jani Tiainen  wrote:
>>
>> Oracle supports comments as well.
>>
>> 7.3.2018 2.52 ip. "vanadium23"  kirjoitti:
>>
>>> Hello, fellows.
>>>
>>> There was once a proposal about ability to add comments to table/columns 
>>> in postgres: https://code.djangoproject.com/ticket/18468
>>> I re-read discussion in ticket, and it has ended with another proposal, 
>>> that now migrations is within Django, so we can add this feature.
>>>
>>> I want to make it as a third-party libs, but has come to thought that 
>>> need to extend SchemaEditor, but it's not good for maintainability.
>>> Also, not only postgres has this feature, but also MySQL.
>>>
>>> Motivation for this feature is that some sort of users can view code 
>>> comments without access to VCS of the project.
>>>
>>> -- 
>>> You received this message because you are subscribed to the Google 
>>> Groups "Django developers (Contributions to Django itself)" group.
>>> To unsubscribe from this group and stop receiving emails from it, send 
>>> an email to django-develop...@googlegroups.com.
>>> To post to this group, send email to django-d...@googlegroups.com.
>>> Visit this group at https://groups.google.com/group/django-developers.
>>> To view this discussion on the web visit 
>>> https://groups.google.com/d/msgid/django-developers/484264c0-b7d1-4264-b8b5-8857847f6b53%40googlegroups.com
>>>  
>>> 
>>> .
>>> For more options, visit https://groups.google.com/d/optout.
>>>
>>
>> -- 
>> You received this message because you are subscribed to the Google Groups 
>> "Django developers (Contributions to Django itself)" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to django-develop...@googlegroups.com.
>> To post to this group, send email to django-d...@googlegroups.com.
>> Visit this group at https://groups.google.com/group/django-developers.
>> To view this discussion on the web visit 
>> https://groups.google.com/d/msgid/django-developers/CAHn91oesnBvY-mWXoHWa3sf8Vri%2B1qwRHat1yF%2B8Sdm0pU-LhQ%40mail.gmail.com
>>  
>> 
>> .
>> For more options, visit https://groups.google.com/d/optout.
>>
>>
>>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/9ff6c9f5-045e-4bc7-8331-060471b012e2%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: On adding comments to database schema

2018-03-11 Thread vanadium23
That's good. I'll prepare the patch for review.

суббота, 10 марта 2018 г., 10:27:30 UTC+3 пользователь Aymeric Augustin 
написал:
>
> I think we could implement that feature.
>
> (Yes I changed my mind from six years ago when I wontfix'd the ticket.)
>
> -- 
> Aymeric.
>
>
>
> On 8 Mar 2018, at 11:00, Jani Tiainen  
> wrote:
>
> Oracle supports comments as well.
>
> 7.3.2018 2.52 ip. "vanadium23"  
> kirjoitti:
>
>> Hello, fellows.
>>
>> There was once a proposal about ability to add comments to table/columns 
>> in postgres: https://code.djangoproject.com/ticket/18468
>> I re-read discussion in ticket, and it has ended with another proposal, 
>> that now migrations is within Django, so we can add this feature.
>>
>> I want to make it as a third-party libs, but has come to thought that 
>> need to extend SchemaEditor, but it's not good for maintainability.
>> Also, not only postgres has this feature, but also MySQL.
>>
>> Motivation for this feature is that some sort of users can view code 
>> comments without access to VCS of the project.
>>
>> -- 
>> You received this message because you are subscribed to the Google Groups 
>> "Django developers (Contributions to Django itself)" group.
>> To unsubscribe from this group and stop receiving emails from it, send an 
>> email to django-develop...@googlegroups.com .
>> To post to this group, send email to django-d...@googlegroups.com 
>> .
>> Visit this group at https://groups.google.com/group/django-developers.
>> To view this discussion on the web visit 
>> https://groups.google.com/d/msgid/django-developers/484264c0-b7d1-4264-b8b5-8857847f6b53%40googlegroups.com
>>  
>> 
>> .
>> For more options, visit https://groups.google.com/d/optout.
>>
>
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to django-develop...@googlegroups.com .
> To post to this group, send email to django-d...@googlegroups.com 
> .
> Visit this group at https://groups.google.com/group/django-developers.
> To view this discussion on the web visit 
> https://groups.google.com/d/msgid/django-developers/CAHn91oesnBvY-mWXoHWa3sf8Vri%2B1qwRHat1yF%2B8Sdm0pU-LhQ%40mail.gmail.com
>  
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>
>
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/f37ea3f6-1f9c-4699-8feb-c85d024130e2%40googlegroups.com.
For more options, visit https://groups.google.com/d/optout.


Re: On adding comments to database schema

2018-03-09 Thread Aymeric Augustin
I think we could implement that feature.

(Yes I changed my mind from six years ago when I wontfix'd the ticket.)

-- 
Aymeric.



> On 8 Mar 2018, at 11:00, Jani Tiainen  wrote:
> 
> Oracle supports comments as well.
> 
> 7.3.2018 2.52 ip. "vanadium23"  > kirjoitti:
> Hello, fellows.
> 
> There was once a proposal about ability to add comments to table/columns in 
> postgres: https://code.djangoproject.com/ticket/18468 
> 
> I re-read discussion in ticket, and it has ended with another proposal, that 
> now migrations is within Django, so we can add this feature.
> 
> I want to make it as a third-party libs, but has come to thought that need to 
> extend SchemaEditor, but it's not good for maintainability.
> Also, not only postgres has this feature, but also MySQL.
> 
> Motivation for this feature is that some sort of users can view code comments 
> without access to VCS of the project.
> 
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to django-developers+unsubscr...@googlegroups.com 
> .
> To post to this group, send email to django-developers@googlegroups.com 
> .
> Visit this group at https://groups.google.com/group/django-developers 
> .
> To view this discussion on the web visit 
> https://groups.google.com/d/msgid/django-developers/484264c0-b7d1-4264-b8b5-8857847f6b53%40googlegroups.com
>  
> .
> For more options, visit https://groups.google.com/d/optout 
> .
> 
> -- 
> You received this message because you are subscribed to the Google Groups 
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an 
> email to django-developers+unsubscr...@googlegroups.com 
> .
> To post to this group, send email to django-developers@googlegroups.com 
> .
> Visit this group at https://groups.google.com/group/django-developers 
> .
> To view this discussion on the web visit 
> https://groups.google.com/d/msgid/django-developers/CAHn91oesnBvY-mWXoHWa3sf8Vri%2B1qwRHat1yF%2B8Sdm0pU-LhQ%40mail.gmail.com
>  
> .
> For more options, visit https://groups.google.com/d/optout 
> .

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/6B4870F7-F4B4-46D0-9102-EB840B3BD63C%40polytechnique.org.
For more options, visit https://groups.google.com/d/optout.


Re: On adding comments to database schema

2018-03-08 Thread Jani Tiainen
Oracle supports comments as well.

7.3.2018 2.52 ip. "vanadium23"  kirjoitti:

> Hello, fellows.
>
> There was once a proposal about ability to add comments to table/columns
> in postgres: https://code.djangoproject.com/ticket/18468
> I re-read discussion in ticket, and it has ended with another proposal,
> that now migrations is within Django, so we can add this feature.
>
> I want to make it as a third-party libs, but has come to thought that need
> to extend SchemaEditor, but it's not good for maintainability.
> Also, not only postgres has this feature, but also MySQL.
>
> Motivation for this feature is that some sort of users can view code
> comments without access to VCS of the project.
>
> --
> You received this message because you are subscribed to the Google Groups
> "Django developers (Contributions to Django itself)" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to django-developers+unsubscr...@googlegroups.com.
> To post to this group, send email to django-developers@googlegroups.com.
> Visit this group at https://groups.google.com/group/django-developers.
> To view this discussion on the web visit https://groups.google.com/d/
> msgid/django-developers/484264c0-b7d1-4264-b8b5-
> 8857847f6b53%40googlegroups.com
> 
> .
> For more options, visit https://groups.google.com/d/optout.
>

-- 
You received this message because you are subscribed to the Google Groups 
"Django developers  (Contributions to Django itself)" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-developers+unsubscr...@googlegroups.com.
To post to this group, send email to django-developers@googlegroups.com.
Visit this group at https://groups.google.com/group/django-developers.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-developers/CAHn91oesnBvY-mWXoHWa3sf8Vri%2B1qwRHat1yF%2B8Sdm0pU-LhQ%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.