MyrkviAtWork
/
n58-kantine
1
0
Fork 0
Code Issues Pull Requests 1 Packages Projects Releases 1 Wiki Activity

Add a location option to the program, for using with other canteens than the one in N58 #1

Open
myrkvi wants to merge 3 commits from feature/add-location-option into main
pull from: feature/add-location-option
merge into: MyrkviAtWork:main
Conversation 2 Commits 3 Files Changed 3 +20 -14
myrkvi commented 2023-08-31 17:06:35 +02:00
Owner
Copy Link

(Submitted by Håkon H. Lerring)

Hei,

Vi får også Toma som kantine så jeg utvidet ditt lille program med optional "location"-argument så vi kan bruke det her på Jonsvoll.

Jeg fjernet også simd-støtten i tl siden dette ikke er helt støttet i stable, men du står fritt til å ignorere dette.

Adds a location parameter/argument to the program, defaults to N58 if not specified.
Removes the SIMD feature of the tl crate, as it is not fully supported in stable Rust.

(Submitted by Håkon H. Lerring) > Hei, > > Vi får også Toma som kantine så jeg utvidet ditt lille program med optional "location"-argument så vi kan bruke det her på Jonsvoll. > > Jeg fjernet også simd-støtten i tl siden dette ikke er helt støttet i stable, men du står fritt til å ignorere dette. Adds a location parameter/argument to the program, defaults to N58 if not specified. Removes the SIMD feature of the `tl` crate, as it is not fully supported in stable Rust.
myrkvi added 2 commits 2023-08-31 17:06:36 +02:00
2f32c57c55 Add support for multiple locations 
Signed-off-by: Vegard Berg <mail@myrkvi.com>
c296dd7731 Remove simd since it's not supported in stable rust 
Signed-off-by: Vegard Berg <mail@myrkvi.com>
hakon commented 2023-09-01 07:20:02 +02:00
First-time contributor
Copy Link

I see that I accidentally changed the SOUP_URL to MENU_URL in both instances. See attached patch.

I see that I accidentally changed the SOUP_URL to MENU_URL in both instances. See attached patch.
0003-Soup-items-were-fetched-from-menu-url.patch
1.6 KiB
myrkvi added 1 commit 2023-09-11 15:43:30 +02:00
0141e9de76 Soup items were fetched from menu url 
Signed-off-by: Vegard Berg <mail@myrkvi.com>
myrkvi commented 2023-09-11 15:44:52 +02:00
Author
Owner
Copy Link

I see that I accidentally changed the SOUP_URL to MENU_URL in both instances. See attached patch.

@hakon Har dessverre vært sjuk og litt borte fra teknologien, så ble litt sent. Men takker! Lagt inn nå!

> I see that I accidentally changed the SOUP_URL to MENU_URL in both instances. See attached patch. @hakon Har dessverre vært sjuk og litt borte fra teknologien, så ble litt sent. Men takker! Lagt inn nå!
This pull request can be merged automatically.
You are not authorized to merge this pull request.
You can also view command line instructions.

Step 1:

From your project repository, check out a new branch and test the changes.
git checkout -b feature/add-location-option main
git pull origin feature/add-location-option

Step 2:

Merge the changes and update on Gitea.
git checkout main
git merge --no-ff feature/add-location-option
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: MyrkviAtWork/n58-kantine#1
No description provided.
Delete Branch "feature/add-location-option"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?