- Issue created by @the_g_bomb
- Merge request !501Issue #3505488: Improve instructions for the AI Installer. β (Merged) created by the_g_bomb
- πΊπΈUnited States phenaproxima Massachusetts
One question but otherwise this seems okay to me.
- π¬π§United Kingdom the_g_bomb
Hey, I went with that to make it obvious the user had to go to a different site then and they would know where to go later. This might be their first introduction to Open AI or Anthropic and they may not know the URLs.
It is also possible that they know the tools but don't know where to get the API Keys.
- π¬π§United Kingdom the_g_bomb
As an FYI, this is what it looks like, which I don't think looks that clunky as it isn't duplicated.
- πΊπΈUnited States phenaproxima Massachusetts
It's not to my taste, but honestly, I also don't feel anywhere near strongly enough to die on this hill. :)
So, if the AI leads like it, I'm fine with it!
- π¬π§United Kingdom the_g_bomb
I changed it, same reasons as yours (hills etc)
- π¬π§United Kingdom yautja_cetanu
Can we add (There are loads of warnings for this but its caused so many issues)
Note: You will need to be on a paid Tier for the AI features to work.
I agree that this isn't an ideal solution but it's better than it currently is until we figure out longer term solutions (in browser models, some default that doesn't need an API key)
- π¬π§United Kingdom the_g_bomb
Added the Note as requested. I even got AI to help tidy up the wording.
- πΊπΈUnited States thejimbirch Cape Cod, Massachusetts
Suggestions added. Looks good, moving to RTBC.
- πΊπΈUnited States phenaproxima Massachusetts
I don't see any harm in backporting this to 1.0.x but it does mean we'll want to update the constraints for drupal_cms_ai in that branch.
-
phenaproxima β
committed cd0c3f8f on 1.0.x authored by
the_g_bomb β
Issue #3505488 by the_g_bomb, phenaproxima, thejimbirch, yautja_cetanu:...
-
phenaproxima β
committed cd0c3f8f on 1.0.x authored by
the_g_bomb β
-
phenaproxima β
committed 96319915 on 1.x authored by
the_g_bomb β
Issue #3505488 by the_g_bomb, phenaproxima, thejimbirch, yautja_cetanu:...
-
phenaproxima β
committed 96319915 on 1.x authored by
the_g_bomb β
- πΊπΈUnited States phenaproxima Massachusetts
Whoops, guess I accidentally committed this to 1.0.x first. Maybe that's what the MR targeted. Which is wrong, but not the end of the world. Perhaps in GitLab I can prevent MRs from being opened directly against release branches.