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 new image page #3

Closed
24 tasks
jimcky opened this issue May 30, 2018 · 7 comments
Closed
24 tasks

Add new image page #3

jimcky opened this issue May 30, 2018 · 7 comments

Comments

@jimcky
Copy link
Collaborator

jimcky commented May 30, 2018

Error message:

https://docs.google.com/spreadsheets/d/1NAbJdl7Pti4D\_lhzBd1A\_C9pC25L4nlouZVdI28hvOU/edit#gid=0

  • Can be accessed when user goes to a certain URL

  • Checks if metamask wallat app is available and is linked to a recognized likecoin user

  • When file is uploaded, shows a preview of the image

  • User can use a file picker the upload file

  • When user selected the file, validate the image and check if it matches the requirement of puttyimage assets
    Size, file format etc...

  • Reject files that doesn't match the requirements and shows an error message

  • Shows form for entering information for the image

    • Image description (optional)

    • tags (must enter at least 1)

    • Image license (must select)

      • CC0
  • Shows disclaimer and terms to link

  • require users to explicitly click the "I accept the terms and conditions of puttyimage" checkbox before the image can be uploaded

  • Disable the button if user haven't checked the accept terms checkbox

  • When user clicks the Upload image now button, validates the input in the form, shows error message next to field where necessary

  • Checks for similar image and warn user they should only upload original image

  • Check if identical images had been uploaded before and warn user that they should only upload original images they owned

  • only support png & jpg

  • no minimal size, max size allowed = 10Mb

  • require user to choose a license for the image being uploaded, can be one of 4
    cc0,

    cc-by,

    cc-by-sa,

    cc-by-nd

  • shows a hint under the license input: "Learn more about %license name", link to corresponding page of explaining the license
    cc0

    cc-by,

    cc-by-sa,

    cc-by-nd

  • user must input at least one tag

  • tag must be 2 - 20 characters

  • up to 5 tags can be added

  • must select a license to upload

  • if all checks pass, call API, upload image to IPFS and register the record on blockchain

  • generate and register embed code once upload is completed

  • When image upload failed, display an error message

Screen Shot 2018-06-28 at 18.03.07.png (https://oursky.aha.io/attachments/6572074087508617256/token/a68f6ef639d9b79785ad998c922fab537db2cd808c830c1ca45376c535a17d1e.download?size=original)

Created from Aha! https://oursky.aha.io/features/PUT-15

@williamchong
Copy link
Member

@nwingt @mckingho please try to actually work on this card and see what kind of problem you might encounter

@mckingho
Copy link
Contributor

mckingho commented Jun 6, 2018

Checks for similar image and warn user they should only upload original image
should not be completed in phase 1.

@nwingt
Copy link
Member

nwingt commented Jun 6, 2018

@williamchong007 Is Checks if MetaMask wallat app is available and is linked to a recognized likecoin user related to #8?

@williamchong
Copy link
Member

probably, also it involved calling a like.co api that does not exist yet @nwingt

@nwingt
Copy link
Member

nwingt commented Jun 6, 2018

@williamchong007 Maybe refactor EthHelper into common module?

@williamchong
Copy link
Member

no idea on how to do that elegantly but it would be nice @nwingt

@rickmak
Copy link
Collaborator

rickmak commented Jun 6, 2018

Please add in progress label to avoid duplicate effort on the same issue.

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

5 participants