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

Post Intent #1168

Merged
merged 15 commits into from May 14, 2024
Merged

Post Intent #1168

merged 15 commits into from May 14, 2024

Conversation

flipperbw
Copy link
Contributor

@flipperbw flipperbw commented May 9, 2024

what this does

  • add intent post page
  • remove orchestrator urls from transmitter, add stage to handledrop
  • add max video size
  • handle external meta data
  • ability to add string tags to post
  • add column "domain" for Tool table
  • global post tag limit
  • add tool on post image creation if metadata is present
  • add UI display for external image meta

Testing instructions:

Example link: https://dev.civitai.com/intent/post?mediaUrl=https://files.catbox.moe/h675id.png&detailsUrl=https://dev.civitai.com/api/test&title=testing%20post&description=hello&tags=whatever,eboy

Try the following:

  • bad params (missing mediaUrl, url that isn't valid, etc)
  • url that isnt media (google.com)
  • url that is media but has cors issues
  • url that is too big (500mb image)

After creating:

  • details url whose domain is not allowed
  • details url that doesnt work
  • details url with bad params
  • make sure tags are good, other things get set (title)
  • check that valid details url was parsed successfully
  • check external data with no image metadata, with image meta, no external, etc

For each:

  • do image and video

- remove orchestrator urls from transmitter, add stage to handledrop
- add max video size
- handle external meta data
- ability to add string tags to post
- global post tag limit
- accept any json primitive for details key-value
- add tool on post image creation if metadata is present
@flipperbw flipperbw merged commit ae97e11 into main May 14, 2024
1 check passed
@flipperbw flipperbw deleted the feature/post-intent branch May 14, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant