flybywiresim / discord-bot Goto Github PK

Hello,

a new checklist is available here: Manuals/pdf/A32NX Documentation/FBW A32NX Checklist.pdf

Thanks!

Please reference the following PR for changes. I will post the expect links into this issue at a later time when the PR is in review state.

• flybywiresim/docs#317

Moved Pages

URL Links to follow

• A.Floor Command/Page - https://docs.flybywiresim.com/pilots-corner/advanced-guides/protections/afloor/
• Fix Info Command/Page (if there is one) - https://docs.flybywiresim.com/pilots-corner/advanced-guides/flight-planning/fixinfo/

-- See mav's comment below + just check the linked PR for the merged symbol for when actually merged

Bug type

Wrong data or information

Describe the bug

The Airframe command points to a place in the doc that highlights the info has moved. Bot should use the correct link as well.

Expected behavior

Use https://docs.flybywiresim.com/fbw-a32nx/feature-guides/simbrief/#simbrief-airframe

Steps to reproduce

1. Execute the .airframe command

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

straks#7240

This is a list of missing and wanted commands for the bot. Please comment with any commands you wish to see added.

• Sticky

A command where people can do .da or .dh since most people probably don't know which is baro and radio

radio = DH
baro = DA

Bug type

Wrong data or information

Describe the bug

Commands linking to simbridge information have to be checked and updated where necessary after a doc change recently

Expected behavior

Correct links work

Steps to reproduce

1. Run .remotemcdu
2. Click the link

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

straks#7240

Feature type

Support information

Description

Due to the ECONNRESET error being the #1 reported issue in the installer support channel at the moment a command outlining basic troubleshooting steps would reduce the strain this has on support.

References (optional)

No response

Additional info (optional)

I am already working on this so feel free to assign me to this issue :)

Discord Username (optional)

ExampleWasTaken#0886

Feature type

Support information

Description

Help users to check their brake inputs and verify if they are constantly active or not at all.

.brakecheck is probably the best choice with an alias of .brake

References (optional)

https://discord.com/channels/738864299392630914/838062729398976522/1075782094468436080

Additional info (optional)

No response

Discord Username (optional)

straks#7240

Feature type

Utility or complex functionality

Description

Append the following to every message sent by Shomas

from Shomas, Real A320 FO with 4000+ hours and LOWI Certified, soon to be Captain

References (optional)

• Shomas

Additional info (optional)

• Shomas

Discord Username (optional)

jakuski#9191

Bug type

Utility or complex functionality issue

Describe the bug

Since Discord have removed the need for a discriminator in their update to Discord usernames, the 'Executed By' text at the bottom of some commands still puts "#0" at the end despite the username not having any discriminator anymore.

Example Discord Message - https://discord.com/channels/738864299392630914/740722295009706034/1115821195917525032

My username is now just jason_91 in Discord, since being updated.

Expected behavior

Username should show the discriminator if the username has one, otherwise it should not show "#0" if there isn't one.

Steps to reproduce

1. Use Bot Command that displays "Executed By" text when used

References (optional)

Discord Blog Post on Username Changes - https://discord.com/blog/usernames

Additional info (optional)

No response

Discord Username (optional)

jason_91

The command would provide a link and a short description of the abbreviations page in FBW docs.

Requires the PR below to be merged:

• flybywiresim/docs#524

Final hyperlink would be https://docs.flybywiresim.com/pilots-corner/abbreviations/

Bug type

Wrong data or information

Describe the bug

In the command .exp there are two links to deleted discord channels.

Expected behavior

Correct links or remove them.

Steps to reproduce

.exp

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

Slein

Update the hyperlink used in .calibrate to better future proof against any potential changes to the established hyperlink. (This would be most notable when flyPadOS3 documentation get's merged).

The link below is valid and provides an internal redirect on the docs website to wherever the throttle calibration page is:
https://docs.flybywiresim.com/throttle

Missing ADIRS command from the previous implementation.

Feature type

Support information

Description

A command that will show some information regarding debugging simbridge. Suggested text:

Title: Understanding why SimBridge is crashing

SimBridge will typically crash moments after it has started due to a bad installation of SimBridge, missing modules or some other issue. In order to assist you further we will need to see your SimBridge logs.

- Open the folder where SimBridge is installed
- Open a terminal window in that location (Hold `SHIFT` and right click on the empty space of the folder and select "Open Windows Terminal" or "Open Command Prompt" or "Open Powershell Window" 
- Type in `fbw-simbridge.exe` and press enter
- Screenshot the output of the console

Recommend image (would suggest someone else to pick one as mine is bloated)

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

alepouna

Feature type

Aircraft information

Description

Provide a link to the GSX Integration documentation

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

straks#7240

Feature type

Utility or complex functionality

Description

modify existing permissions per the comments in #405, as well as potential changes for other commands

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

pererry#8489

The current request dependency is deprecated and a switch to node-fetch would be cleaner and is still maintained.

https://www.npmjs.com/package/node-fetch

Here's a sample of what happens (in our #discord-bot channel)

The image is missing even though the command has a URL in place.

Could be an issue with using the media.discordapp vs cdn.discordapp link as discussed further in the channel as well.

Would still need to be fixed however :) -- thanks!

Quantities and units should be seperated by a space (non-breaking ideally, but I'm not sure that's possible for discord). Additionally it would be ideal to give the runway dimensions in SI units as well as classical units.

Potential patch (I don't have a test discord nor the desire to spend that much time on this so I didn't make a PR sorry 😆 ):

diff --git a/src/commands/utils/station.ts b/src/commands/utils/station.ts
index 3b99ada..c24b4d4 100644
--- a/src/commands/utils/station.ts
+++ b/src/commands/utils/station.ts
@@ -25,7 +25,9 @@ export const station: CommandDefinition = {
             const stationReport = JSON.parse(body);

             const runwayIdents = stationReport.runways.map((runways) => {
-                return `**${runways.ident1}/${runways.ident2}:** ${runways.length_ft}ft x ${runways.width_ft}ft`;
+                return `**${runways.ident1}/${runways.ident2}:** \
+                ${Math.round(runways.length_ft * 0.3048)} m/${runways.length_ft} ft x \
+                ${Math.round(runways.width_ft * 0.3048)} m/${runways.width_ft} ft`;
             });

             const stationEmbed = makeEmbed({
@@ -35,9 +37,9 @@ export const station: CommandDefinition = {
                     `**Name:** ${stationReport.name}`,
                     `**Country:** ${stationReport.country}`,
                     `**City:** ${stationReport.city}`,
-                    `**Latitude:** ${stationReport.latitude}`,
-                    `**Longitude:** ${stationReport.longitude}`,
-                    `**Elevation:** ${stationReport.elevation_m}m/${stationReport.elevation_ft}ft`,
+                    `**Latitude:** ${stationReport.latitude}°`,
+                    `**Longitude:** ${stationReport.longitude}°`,
+                    `**Elevation:** ${stationReport.elevation_m} m/${stationReport.elevation_ft} ft`,
                     ,
                     `**Runways (Ident1/Ident2: Length x Width):**`,
                     `${runwayIdents.toString( ).replace(/,/g,"\n")}`,

Bug type

Wrong data or information

Describe the bug

The United States is now available only with a paid subscription. I suggest removing the country specific remark on the bot message and replacing it with something more generic. Solving ECONNRESET issues seems to work just as well when connecting to other countries.

Edit: The United States is available in the free plan but the location you will end up with is very random since it's up to Proton to decide what's better.

Expected behavior

Updated command information

Steps to reproduce

N/A

References (optional)

Additional info (optional)

No response

Discord Username (optional)

ccute

Feature type

Utility or complex functionality

Description

Add a .TAF command which gets TAF just like .metar

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

No response

Feature type

Utility or complex functionality

Description

Some commands get flooded by users in certain situations, it could be useful to have a rate limiter implemented that prevents a command from being used too often within a single channel.

Proposal for the feature is:

• Support rate limiting a command (based on time: set to a certain time interval, a command can only be used once within the time interval)
• Introduce mod command to manage (set, unset) a rate limit for a command, can be any command (automatically matches the aliased)
• Mod commands should be blocked from rate limiting

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

straks#7240

Feature type

Utility or complex functionality

Description

Currently the taf(#467 ), metar and station commands all use different environmental variables, which all have the same value specified in the .env file. In my opinion they all could use the same variable, which would simplify documentation and deployment to devolepement enviroments.

References (optional)

TAF_TOKENs are from #467

Additional info (optional)

No response

Discord Username (optional)

Lauvaan#3330

Feature type

Utility or complex functionality

Description

Feature request: Incorporate a translation layer between the existing command executors/permissions and Discord's slash command/message app API.

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

pererry#8489

Not entirely sure why we are not using the full potential of bots with slash commands... Gives a much better UI experience and is easier to use the commands.

Also gives better discord native functions.

Bug type

Wrong data or information

Describe the bug

Links in the versions command all point towards Bunny CDN.

Expected behavior

Should use our new cloudflare links or I believe github in the case of full downloads not from installer.

Steps to reproduce

1. Run command on Discord.
2. Click on "Download Here"the

References (optional)

Additional info (optional)

No response

Discord Username (optional)

Valastiri#8902

https://docs.flybywiresim.com/fbw-a32nx/feature-guides/loading-fuel-weight/

Can steal copy from our #server-announcements for the embed if required.

This link is live.

Feature type

General information

Description

We frequently get asked what languages we use for various aspects of the A32NX. Would be good to have a nice little bot cheat sheet to send to people for reference.

Can also replicate on docs as well.

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

No response

A command which would provide information on free SimBrief accounts being restricted to AIRAC 2003, and problems that arise with navdata and importing flight plans. Potentially also provide links to alternate route lookup/generators.

Potential alternatives to consider include:
http://rfinder.asalink.net/free/
https://flightplandatabase.com/
http://onlineflightplanner.org/

Feature type

Support information

Description

The control settings in MSFS for the TCA throttle quadrant requires removing certain settings and to properly set up the TCA addons so that it doesn't interfere.

Focus is mostly on flaps and spoilers that would get triggered in case of pause/unpause if they are bound, but the addons aren't present.

References (optional)

https://docs.flybywiresim.com/fbw-a32nx/a32nx-api/hardware/?h=tca#thrustmaster-tca-throttle-add-on-tca-quadrant-add-on-airbus-edition
https://ts.thrustmaster.com/download/pictures/Mappings/Mapping-TCAQuadrant_And_AddOn_MFS.png

Additional info (optional)

No response

Discord Username (optional)

straks#7240

Feature type

Support information

Description

Requesting a support command that will show some template text about the remote efb.

Recommending this text: "The EFB currently can not be reviewed remotely, but it is a planned feature.

You can however, use an app like Spacedesk to create a virtual screen on your tablet/phone/pc and then de-attach the EFB from MSFS by holding ALTGR and clicking on the EFB. You can now drag window to your virtual screen. Please note that this solution will degrade your performance slightly as your computer needs to render and process the new virtual screen.
"

This will be handy as this topic is being actively brought up and would provide a much more clear answer easily!

References (optional)

No response

Additional info (optional)

No response

Discord Username (optional)

Alepouna🌙#9824

Bug type

Wrong data or information

Describe the bug

Outdated info according to Donbikes. Requests we update the command in the last line to say that we are working on our own takeoff performance calculator.

Expected behavior

New updated command

Steps to reproduce

n/a

References (optional)

Support ops mention - here

Additional info (optional)

No response

Discord Username (optional)

valastiri