Skip to content

Fixed the option for video upload#71

Open
TEAMCHRIST wants to merge 1 commit into
FOSSEE:masterfrom
TEAMCHRIST:video_upload_fixed
Open

Fixed the option for video upload#71
TEAMCHRIST wants to merge 1 commit into
FOSSEE:masterfrom
TEAMCHRIST:video_upload_fixed

Conversation

@TEAMCHRIST
Copy link
Copy Markdown

@TEAMCHRIST TEAMCHRIST commented Oct 4, 2019

  • changes are made in the edit proposal page to show video upload button when the reviewer gives permission
  • added a button for reviewer to allow video upload
  • changes in the models.py file to give video upload permission

Closes #40

Copy link
Copy Markdown

@prathamesh920 prathamesh920 left a comment

Choose a reason for hiding this comment

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

Generic comments, applies to all the PRs

  1. Write Python code as per PEP 8 standards
  2. Remove all the commented code
  3. Have slightly long description for the PR
  4. Short PRs preferred as it makes the review process easy and quick
  5. Try to avoid inline CSS

@TEAMCHRIST TEAMCHRIST changed the title Fixed the requirements for video upload Fixed the option for video upload Oct 18, 2019
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.

Modify Submit Comment button Action

3 participants