viktorvav99 / upland-travel-bot

Basic bot for finding the best route for travel in Upland.

Hey!

About me

🎓 I’m an Informatics student with a passion for technology. I’m pursuing a Bachelor of Science degree at Halmstad University with a focus on Digital Design and Innovation, where I learn how to create and improve digital products and services. Besides studying, I also work part-time as an Open Source Community Engineer at Electricity Maps, a company that provides real-time data on the carbon footprint of electricity. There, I contribute to various aspects of the project, such as data parsing, frontend development, and continuous integration.

💻 I have skills in Python, JavaScript, TypeScript, HTML, CSS, SCSS and more. I’m especially interested in optimizing apps for size and performance, especially Single Page JavaScript apps that need to be fast and lightweight.

In my free time, I love to explore nature by hiking or on my mountain bike or on my skis, depending on the season or just watching a nice sci-fi show or movie with some popcorn and relax.

Goal

🎉 My goal is to make the world a better place through code and to help others do the same.

upland-travel-bot's Issues

Filter out duplicate routes and pick the cheapest one.

Fix send count when switching from international to national flights in `/travel`

Migrate to ESM and TypeScript

Migrate the whole codebase to ESM and TypeScript.

Todo:

• #6
• #9

Update `/travel` reply to use embed

Update the /travel command to reply with a embed instead of plaintext.

Migrate to TypeScript

Migrate to TypeScript.

Add Array.groupBy() later when it is stable.

    //TODO: Add Array.groupBy() later when it is stable.

Upland-Travel-Bot/src/commands/cmProps.ts

Lines 35 to 50 in 2a7d8ac

	props = Object.keys(APIData.props)
	.map(
	(key) =>
	APIData.props[key] as {
	id: string;
	city: string;
	address: string;
	}
	)
	.map((propData) => {
	return {
	id: propData.id,
	address: propData.address,
	city: propData.city.split(',')[0],
	};
	});

Make the responseArray generation dynamic

//TODO: Make the responseArray generation dynamic

Upland-Travel-Bot/src/commands/cmProps.ts

Lines 93 to 96 in 2a7d8ac

	if (responseArray1.join('\n').length + property.length <= 1024) {
	responseArray1.push(property);
	} else if (responseArray2.join('\n').length + property.length <= 1024) {
	responseArray2.push(property);

Migrate to ESM

Migrate all JS code to ESM.

Currently blocked by bug in discordjs/builders

Remove node-fetch once fetch is included in node.

import fetch from 'node-fetch'; //TODO: Remove node-fetch once fetch is included in node.

Upland-Travel-Bot/src/commands/cmProps.ts

Line 1 in 2a7d8ac

import fetch from 'node-fetch';

Add info command

#1 Filter out duplicate routes and pick the cheapest one.

Upland-Travel-Bot/travel.js

Lines 112 to 117 in ecc73da

	// TODO: #1 Filter out duplicate routes and pick the cheapest one.
	foundPath.forEach((Node) => {
	nextNode = Node.id;
	links = Node.links.unique();
	links.unique().forEach((Link) => {
	if (Link.toId == nextNode && (Link.fromId == previousNode || '')) {

---

This issue was generated by todo based on a TODO comment in ecc73da. It's been assigned to @VIKTORVAV99 because they committed the code.