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

v4.11 #1117

Merged
merged 13 commits into from
May 15, 2024
Merged

v4.11 #1117

merged 13 commits into from
May 15, 2024

Commits on Apr 13, 2024

  1. Java RSS Feed (#1037)

    * wip: initial commit
    
    * style(spotlessApply): run uncalled task
    
    * fix: register routine in features list
    
    * feat: update config.json.template
    
    * wip(rss-config): change config design
    
    * feat: make targetChannelPattern into a Map
    
    * refactor: don't hardcode number
    
    * feat: improve constructEmbedMessage
    
    * feat: improve the description constructing
    
    * feat: increase MAX_CONTENTS to 300
    
    * wip: changes
    
    * feat(rss-feed): convert polling interval to be configurable
    
    * feat: improve date parsing
    
    Co-authored-by: Ethan McCue <[email protected]>
    
    * feat: update config.json.template
    
    * feat: improve date handling and add embed timestamp
    
    * fix: malformed `config.json.template`
    
    * fix: now correctly finds the latest date
    
    * feat: improve date handling and add embed timestamp
    
    * fix: now correctly finds the latest date
    
    * feat: you can now optionally declare a specific channel for a feed to go to
    
    * finished javadoc todos
    
    * improved embed
    
    * feat(rss-routine): working version
    
    - Added JavaDocs on most parts
    - Simplified the code and cleaned up unused stuff
    
    * feat(rss-routine): increase MAX_CONTENTS to 1000
    
    * refactor(rss-routine): rename to RSSHandlerRoutine
    
    While the original idea was to add a Java news and changes RSS feed,
    this was expanded to all types of RSS feeds, so a more appropriate name
    makes things more clear.
    
    * docs(rss-routine): add JavaDocs for constructor and class
    
    * feat: add @NotNull annotation
    
    * Update RSSHandlerRoutine.java
    
    * fix: reverse feed so it posts in correct order
    
    * refactor: use variable types instead of var
    
    * feat: use fetchAny() instead of fetch()
    
    * fix(rss-handler): remove redundant empty check
    
    Co-authored-by: Ethan McCue <[email protected]>
    
    * refactor(rss-handler): remove AtomicReference usage
    
    Co-authored-by: Ethan McCue <[email protected]>
    
    * resolve a couple issues
    
    * refactor(rss): switch to using a record for the config
    
    * feat: rename to fallbackChannelPattern
    
    * refactor: remove unnecessary throws exception in method signature
    
    * docs: add a few missing parameters and fix typos
    
    * refactor: reduce try-catch scope and added clarifying comment
    
    * perf: use Stream instead of StringBuilder
    
    * refactor: put fallback case into an else statement
    
    * refactor: switch to Map#containsKey() for targetChannelPatterns
    
    * feat: modularize sendRss() method and improve variable names
    
    This also changes the functionality of how new RSS feeds get dealt with
    for the first time. Before this commit, all items would get posted as
    embeds on Discord, and as a result, that would bombard the target
    channel with RSS posts once the routine executes.
    
    This commit changes this behavior by assuming that all RSS posts have
    been posted and it should only consider posting posts newer than the
    registered date.
    
    * refactor: remove star import
    
    * fix: add Objects#requireNonNull() on rssConfig
    
    * changes
    
    * feat: improvements from code reviews
    
    Co-authored-by: Nathan Weisz <[email protected]>
    
    * feat: add DateTimeParseException in signature
    
    * refactor: use .orElseThrow() instead of .get()
    
    * fixed missing coma after rss configuration
    
    Co-authored-by: Tanish <[email protected]>
    
    * fix(rss): feeds support multiple channels
    fix(rss): polling interval is now clear on unit of time
    fix(rss): posting messages now uses forEachOrdered
    
    * Optional<List<TextChannel>> -> List<TextChannel>
    
    * extract item post predicate to seperate function
    
    * various changes from null to optional
    
    ---------
    
    Co-authored-by: christolis <[email protected]>
    Co-authored-by: Ethan McCue <[email protected]>
    Co-authored-by: Tanish <[email protected]>
    4 people authored Apr 13, 2024
    Configuration menu
    Copy the full SHA
    835a951 View commit details
    Browse the repository at this point in the history
  2. quickfix (#1087)

    nateweisz authored Apr 13, 2024
    Configuration menu
    Copy the full SHA
    adc0e16 View commit details
    Browse the repository at this point in the history

Commits on Apr 22, 2024

  1. Proper RSS default config (#1098)

    working config for RSS feed that fits majority of users
    Zabuzard authored Apr 22, 2024
    Configuration menu
    Copy the full SHA
    f109c56 View commit details
    Browse the repository at this point in the history

Commits on May 6, 2024

  1. chore: add safety checks to NoteCommand (#1086)

    chore: add safety checks to NoteCommand
    SimonVerhoeven authored May 6, 2024
    Configuration menu
    Copy the full SHA
    2c3da71 View commit details
    Browse the repository at this point in the history
  2. chore: rename ban history options to be clearer (#1082)

    * chore: rename ban history options to be clearer
    * chore: update ban command period description
    SimonVerhoeven authored May 6, 2024
    Configuration menu
    Copy the full SHA
    b097294 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    2738a98 View commit details
    Browse the repository at this point in the history

Commits on May 12, 2024

  1. fix(rss): logic for acquiring target channels (#1090)

    This commit attempts to change the way a text channel list for RSS feeds
    gets generated, as it would originally not consider any channels that
    matched the fallback channel pattern and it would constantly log a
    warning that would clutter the log channels and skip sending any RSS
    posts as a result.
    
    The method which is responsible for finding the text channels from a
    given RSS feed configuration now focuses on collecting a list of text
    channels that match the target channel pattern from the configuration,
    and if no channels are found, the same collection attempt happens with
    the fallback channel pattern. In case an empty list is still yielded, a
    now-improved and more accurate warning message gets logged.
    christolis authored May 12, 2024
    Configuration menu
    Copy the full SHA
    4e7b9b6 View commit details
    Browse the repository at this point in the history
  2. Remove star imports (#1092)

    chore: remove star imports (resolves #1057)
    SimonVerhoeven authored May 12, 2024
    Configuration menu
    Copy the full SHA
    d198f19 View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    65529a7 View commit details
    Browse the repository at this point in the history

Commits on May 13, 2024

  1. Configuration menu
    Copy the full SHA
    f045ccf View commit details
    Browse the repository at this point in the history
  2. using a better method to retrieve start message for thread (#1106)

    * using a better method to retrieve start message for thread
    
    * replace retrieveMessageById with getStartMessage on threadChannel channel type
    ankitsmt211 authored May 13, 2024
    Configuration menu
    Copy the full SHA
    f8c91a9 View commit details
    Browse the repository at this point in the history

Commits on May 14, 2024

  1. Title generation by ChatGPT for transfer-questions command (#1059)

    * title generation by ChatGPT for transfer-questions command
    
    refactoring
    
    reviewer comments addressed
    
    prompt improved
    
    Update prompt
    
    Co-authored-by: alphaBEE <[email protected]>
    
    removed tags
    
    * refactor prompt message for better titles
    
    ---------
    
    Co-authored-by: alphaBEE <[email protected]>
    vishv843 and ankitsmt211 authored May 14, 2024
    Configuration menu
    Copy the full SHA
    10718cb View commit details
    Browse the repository at this point in the history

Commits on May 15, 2024

  1. Refactor help thread creation workflow (#1112)

    Thread creation flow is more clear and optimized by removing unnecessary calls to discord API
    SquidXTV authored May 15, 2024
    Configuration menu
    Copy the full SHA
    d3a19aa View commit details
    Browse the repository at this point in the history