search

flybywiresim / discord-bot-utils

Utilities Discord Bot created for the FlyByWire Discord server.
https://flybywiresim.com/
GNU Affero General Public License v3.0
5 stars 7 forks source link

feat: move images to new bot repo #92

Open benw202 opened 1 week ago

benw202 commented 1 week ago

Description

Work in Progress:

52 Commands currently have images.

50 images in this PR (53 in total - disregard one for the new profile picture, one command has two images and one is a full quality imaged that isn't used)

Missing 3 images

This PR / adds moves all images from the old bot repo to the new bot repo. Images under A32nx have now been put into a generic Aircraft folder. It is undecided if the Aircraft directory should be further split to A32nx, A380x and general.

Changes need to be made in the database to accommodate this. One change to change the base URL from https://assets.discord.flybywirecdn.com/assets/images/... to https://assets.discord.flybywirecdn.com/utils/assets/images/prefix/.... And one to change any image URL including the a32nx directory to the aircraft directory. I have scripts prepared which need to be run when this PR is merged (Production and Staging). See an example of the change below:

Original URL: https://assets.discord.flybywirecdn.com/assets/images/a32nx/screens.png

New URL: https://assets.discord.flybywirecdn.com/utils/assets/images/prefix/aircraft/screens.png

This PR also updates the profile picture of the production bot

Test Results

N/A

Discord Username

benw8484

benw202 commented 1 week ago

Do not merge label added as DB changes must be made simultaneously