sheldan/sissi

Custom Discord bot for the Austria Discord server.

sissi's Issues

Rename project to Sissi

Add a daily morning report

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Add a daily message that shows in #ratschn the weather in the following cities: Vienna, Graz, Salzburg, Innsbruck. Only temperatures.

Show the last 5 news items from the ORF rss feeder in #zeit-im-bild.

Feature worth and general use

An overview over the weather and some overview about politics. The news articles coming from ORF should not be re-peated.

Suggestions for implementation

No response

`/Meetup list` shows canceled meetup.

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

When using /meetup list meetups that have been canceld are shown as well.

Steps to reproduce

1. Creat meetup.
2. Click Yes im sure button
3. Click Cancel button
4. Use /meetup list command

Expected behaviour

The canceled meetup should not show up in the list of pending meetups.

Actual behaviour

The canceled meetup shows up in the list of pending meetups.

Ability to restrict quote searches to a member and return a random one instead of the first one

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Currently quote search only returns the "first" one it finds, if multiple quotes match for a given text.
Additionally, its not possible to restrict the quotes to one member, but all members are considered.

Feature worth and general use

More convenience when searching for quotes

Suggestions for implementation

No response

Add ics file to meetup message

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

This should be downloadable in some way.

Feature worth and general use

For easier addition to a calendar.

Suggestions for implementation

No response

Add meetup functionality

There should be a functionality to enhance the setup and management of meetups.

A detailed flowchart is available here:

https://cdn.discordapp.com/attachments/460176969108160543/986029851393818634/MEETUP.jpg

Add news post from orf.at to news related channels

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Add the newest posts from the orf.at rss feed.
To posted in #zeit-im-bild and #internationales and #klimakrise, depending on the topic:
Ausland should be posted to #internationales, Umwelt to #klimakrise, Inland to #zeit-im-bild, Science to #tech-science.

Not sure where the rest of the categories should be posted to. #zeit im bild.

Feature worth and general use

Having news posts for informative reasons or purely to cause discussions.

Suggestions for implementation

A dynamic mapping of Subject to channel is preferrable.

Leaving out location returns error

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

If you want to create a meetup but leave out the location, the current create meetup returns an error.

Steps to reproduce

Execute the slash command "/meetup create meetuptime: 1677161487 topic: aslkdfjklasdf"

Expected behaviour

Should create the meetup with the given parameters

Actual behaviour

Shows a nullpointer exception with the following stacktrace

sissibot | Caused by: java.lang.NullPointerException: null sissibot | at java.net.URLEncoder.encode(URLEncoder.java:204) sissibot | at dev.sheldan.sissi.module.meetup.service.management.MeetupManagementServiceBean.createMeetup(MeetupManagementServiceBean.java:45) sissibot | at dev.sheldan.sissi.module.meetup.commands.CreateMeetup.executeSlash(CreateMeetup.java:132)

Meetup title is not properly escaped, leading to potential wrong syntax when using embeds to render a message

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

Meetups with "something" in their title, fail to render, making it impossible to list meetups.

Steps to reproduce

Have a meetup with "something" in its title and try to list all the meetups.

Expected behaviour

The meetups should still display fine

Actual behaviour

Meetup list command fails with:

com.google.gson.JsonSyntaxException: com.google.gson.stream.MalformedJsonException: Unterminated object at line 15 column 18 path $.embeds[0].description

Enable and align moderation features

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

The moderation features from abstracto should be used and customized accordingly.

Feature worth and general use

Better integration of moderation features

Suggestions for implementation

No response

Sometimes the donation notification contains the old donation amount

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

There needs to be a delay between the donation notification and the amount retrieved from the site, as the site does not always keep up to date.

Steps to reproduce

Make a donation and see the notification message

Expected behaviour

Reflect the correct donation amount (or as good as possible)

Actual behaviour

Sometimes the donation amount is the old donation amount

Port modmode

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Add modmode feature to switch the color of a pre-defined role

Feature worth and general use

Necessary for feature completeness.

Suggestions for implementation

No response

Adding organizer to meetup message

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

The organizer should be clearly visible in the meetup message.

Feature worth and general use

Information on who organized the meetup and who can also cancel the meetup. This information is not visible anywhere.

Suggestions for implementation

No response

Include necessary features

The modules to include are:
modmail, remind, webservices, assignable roles, suggestion, utility

Enable experience and join/leave log functionality

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Experience should be ported and configuration should be taken over.
Join/leave log should be enabled as well, and only join/leave from the logging module.

Feature worth and general use

No dependency towards mee6 anymore.

Suggestions for implementation

No response

Updating Debra donation links to 2022

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

The old links need to be updated to fit with the new link
https://www.debra-austria.org/spenden-helfen/eigene-spendenaktion-starten/?discord-fuer-debra-2022.
Also there should be two post targets for the donation notifications

Feature worth and general use

The new page is the correct one to reflect it.

Suggestions for implementation

No response

Meetup enhancements

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Various meetup improvements:

• Show which meetup in meetup notify
• Add an option to define which meetup participant types should be notified
• Possibility to change description
• Add ability to define the location which is then linked via google maps (optional=) via this url https://www.google.com/maps?q=%s

Feature worth and general use

More convenient and more informative output

Suggestions for implementation

No response

Not possible to delete mappings after a news post has been created

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

Once a news post is created, a news category channel mapping cannot be deleted anymore due to foreign keys.

Steps to reproduce

Create news category with source category and channel mapping
Let a news post be created
Delete the news category/mapping

Expected behaviour

It should be deleted

Actual behaviour

Fails with org.springframework.dao.DataIntegrityViolationException: could not execute statement [ERROR: update or delete on table "news_category_channel_mapping" violates foreign key constraint "fk_news_post_news_category_channel_mapping" on table "news_post"
Detail: Key (id)=(3) is still referenced from table "news_post".] [delete from abstracto.news_category_channel_mapping where id=?]; SQL [delete from abstracto.news_category_channel_mapping where id=?]; constraint [fk_news_post_news_category_channel_mapping]

Migration from franz

This migration includes:
warnings, economy, custom commands

Adding parameters for donations command to show latest and highest donations

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Add optional parameters to the donation command to show x highest/latest donations with name and amount.

Feature worth and general use

In case you want to have an overview over donations this command can be used

Suggestions for implementation

It should use the AP instead of HTML parsing.

Support debra donation functionality

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Take the functionality from https://github.com/zinnsoldat91/spendenbot and integrated it into sissi.
It should behave the same and offer the usual configurability of templates and similar.

Feature worth and general use

Support of the debra donation gathering in December.

Suggestions for implementation

No response

Create privacy plicy

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

It should contain a list of information which is stored, if and when its deleted and how to contact in case of a request for removal.

Feature worth and general use

Privacy concerns

Suggestions for implementation

No response

Add MeetupDecision Buttons inklucding ability to change the underlying unix timestamp of the meetup.

General Information

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming features fixes.

Description

Change Buttons from YES, Maybe , No and Cancel to the a different assortment of Buttons for the user/meetup organizer to click. Add Change Time Feature.

Buttons

Overview

• MeetupDecision.YES
• MeetupDecision.MAYBE
• MeetupDecision.NoTime
• MeetupDecision.NotInterested
• MeetupDecision.CANCEL
• MeetupDecision.ChangeTime

MeetupDecision.YES:

The user is attending the meetup and has agreed to the meetup's date, time and location.

Constraint: None

MeetupDecision.MAYBE

The user would like to attend to the meetup, but does not yet know if they can attend the meetup at the given time and date.

Constraint: None

MeetupDecision.NoTime

The user would like to attend the meetup, but can not attend because he does not have time at the set date and time.

Constraint: None

MeetupDecision.NotInterested

The user has no interest in the meetup topic. His decision to not attend the meetup is completely independent of the date and time.

Constraint: None

MeetupDecision.CANCEL

The meetup is canceled.

Constraint: Caller has to be organizer

MeetupDecision.ChangeTime

The UNIX timestamp for the given meetup is changed.

Constraint: Caller has to be organizer

Feature worth and general use

Currently, there is no way to change the underlying UNIX timestamp of the meetup. -> Add Button MeetupDecision.ChangeTime

Split option NO into NoTime and NotInterested.

Suggestions for implementation

MeetupDecision.ChangeTime

MeetupDecision.NotInterested

When the user presses NotInterested it work as a mark as read. Do not show the List of users that selected this option separately. A simple Not Interested: (COUNT OF NOT INTERESTED USERS) should suffice.

Markup of the meetup announcement:

Meetup location length limit is not enforced in slash command

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

It possible to execute a slash command with a location longer than 100 characters, but actually creating the meetup fails.

Steps to reproduce

Create a meetup with a longer than 100 characters location

Expected behaviour

Should not be possible to submit the command, or the command should fail gracefully

Actual behaviour

The command fails with the error message in the logs that the insert into the database was not possible.

Location in meetup ICS file is encoded, that breaks a lot of usage for the file

General Troubleshooting

• I have checked for similar issues on the Issue-tracker.
• I have updated to the latest version
• I have checked the branches or the maintainers' PRs for upcoming bug fixes.

Description

The location in the ICS file for meetups is URL encoded, that leads to it not working normally, because it is not correctly interpreted in most cases.

Steps to reproduce

Create meetup with spaces or äü in the location

Expected behaviour

Should provide the location correctly in the ICS file

Actual behaviour

The location is URL encoded, which breaks google maps for example

Upgrade to Abstracto 1.4