Joachim Tislov
Joachim Tislov
There are still some styling conflicts. Specifically because of `float-right`, `badge(-*)` and a few other classes defined by Bootstrap. Examples (The ones I could find): "pending enrollments" notification does not...
The previously mentioned "conflicts" occur because of the base layer styling in Tailwind.
By the latest commits, it should be OK to merge. I'm certain that these changes resolve every known conflict I have encountered, but I'm not sure if there are more....
Master:  This PR: 
If its storing the enterprise url shouldn't it be called: `QUICKFEED_ENTERPRISE_URL` and is this link available after creating the app? As @JosteinLindhom mentioned [here](https://github.com/quickfeed/quickfeed/pull/1337/files/5b12ce64c5a3f4ab7e412e95ae23c23ffcee7e71#diff-68cbd9d4482ed9e3300855e7eb7e08de7b1b7987e0cecccdc42b5be3c19b8ef4), was the `QUICKFEED_ORGANIZATION_URL` static, since it...
CheckCode duplicates the logic of `CheckError`. Consider calling CheckError instead? ```go func CheckCode(t *testing.T, got, want error) bool { CheckError(t, got, want) if connect.CodeOf(got) != connect.CodeOf(want) { t.Errorf("Expected error code:...
The `CreateNewQuickFeedApp` doesn't return the server. So making the Post request isn't possible? ```go _, err := server.Client().Post(url, "application/json", nil) if err != nil { t.Fatalf("failed to send request: %v",...
> @JoachimTislov @JosteinLindhom @Glattshaved Are there any issues merging this? Yes, `Switch`, `useHistory` and `Redirect` will be outdated and cause these errors, when compiling the frontend: 
Refering to https://github.com/quickfeed/quickfeed/pull/1328#issuecomment-2897632070. I thought [this commit](https://github.com/quickfeed/quickfeed/pull/1344/commits/cd3f1c5b491341e3007e5a276c047def87079589) resolved #1291, but after addtional testing, it appears to not fix the issue. And I also noticed that the box with the test...
The `No automated tests for this assignment` message isn't loaded in the build log div after a refresh, but everything else works!