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

feat: add Effis support #13

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

feat: add Effis support #13

wants to merge 5 commits into from

Conversation

spifory
Copy link
Member

@spifory spifory commented Feb 23, 2023

Summary

This Pull Request implements every bit of functionality for Effis.

Checklist:

  • This PR has been tested
    • The documentation has reflected the changes
    • The code has been linted and formatted
  • This PR fixes an issue
  • This PR adds a new feature
  • This PR adds a breaking change
  • This PR is NOT a code change (e.g. documentation, README, ...)

@spifory spifory added p: high Priority: high - should be worked on as soon as possible t: feature Type: feature - is a feature request/addition labels Feb 23, 2023
@spifory spifory added this to the v1.3.0 milestone Feb 23, 2023
@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Feb 23, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: f67b2ff
Status: ✅  Deploy successful!
Preview URL: https://2847b96b.evangeline.pages.dev
Branch Preview URL: https://feat-support-attachments.evangeline.pages.dev

View logs

@spifory spifory marked this pull request as ready for review February 23, 2023 09:07
@spifory spifory requested a review from EnokiUN February 23, 2023 09:10
}

/**
* Send a message to Eludris. (Alias for {@link sendMessage})
* @param content The content of the message.
*/
async send(content: string): Promise<Message> {
return await this.sendMessage(content)
async send(content?: string, file?: {
Copy link
Member

Choose a reason for hiding this comment

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

why not allow multiple files?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
p: high Priority: high - should be worked on as soon as possible t: feature Type: feature - is a feature request/addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support Effis related functionality
2 participants