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

Add description to project #6258

Closed
wants to merge 1 commit into from
Closed

Conversation

pktiuk
Copy link
Contributor

@pktiuk pktiuk commented Jun 7, 2023

Partially linked with: #3680

Motivation and context

I think it would be very handy to have at least a short description of project available in CVAT.
It could help with organizing and adding context to Projects.

How has this been tested?

I just deployed it with help of the docker-compose.dev.yml you provided.

For now this is just a draft, because before finishing this feature I would like to get feedback from cvat maintainers.

image

Checklist

  • I submit my changes into the develop branch
  • I have added a description of my changes into the CHANGELOG file
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • I have linked related issues (see GitHub docs)
  • I have increased versions of npm packages if it is necessary
    (cvat-canvas,
    cvat-core,
    cvat-data and
    cvat-ui)

Other things to do:

  • Fix checks
  • Allow searching descriptions

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.

@@ -292,6 +292,7 @@ class Meta:

class Project(models.Model):
name = SafeCharField(max_length=256)
description = SafeCharField(max_length=1400, default="")
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is arbitrary length I propose, but maybe you think it would be good to change it.
I am also not sure which field type you would recommend SafeCharField or regular CharField

@bsekachev
Copy link
Member

Hi, @pktiuk

Thank you for your contribution. But we are now working on more powerful project/task description.

#6191

@pktiuk pktiuk closed this Jun 7, 2023
@pktiuk
Copy link
Contributor Author

pktiuk commented Jun 7, 2023

Good to know.
That's a shame it was not linked with the issue #3680 earlier.

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

Successfully merging this pull request may close these issues.

2 participants