36 fix logbook edit view error #37

Merged
noahspannbauer merged 4 commits from 36-fix-logbook-edit-view-error into main 2025-02-02 20:51:42 -05:00
6 changed files with 53 additions and 5 deletions

28
.github/workflows/changes.yaml vendored Normal file
View File

@@ -0,0 +1,28 @@
name: Changes
on:
workflow_call:
outputs:
api:
value: ${{ jobs.changes.outputs.api }}
app:
value: ${{ jobs.changes.outputs.app }}
jobs:
changes:
name: filter
runs-on: ubuntu-latest
outputs:
api: ${{ steps.filter.outputs.api }}
app: ${{ steps.filter.outputs.app }}
steps:
- uses: actions/checkout@v3
- uses: dorny/paths-filter@v3
id: filter
with:
initial-fetch-depth: 2
filters: |
api:
- 'api/**'
app:
- 'app/**'

View File

@@ -5,8 +5,14 @@ on:
- main
jobs:
changes:
uses: ./.github/workflows/changes.yaml
build-api:
if: ${{ needs.changes.outputs.api == 'true' }}
name: build-api
needs:
- changes
uses: ./.github/workflows/api_build.yaml
with:
environment_name: test
@@ -15,7 +21,9 @@ jobs:
deploy-api:
name: deploy-api
needs: build-api
needs:
- changes
- build-api
uses: ./.github/workflows/deploy.yaml
with:
app_name: flying-app
@@ -24,7 +32,10 @@ jobs:
secrets: inherit
build-app:
if: ${{ needs.changes.outputs.app == 'true' }}
name: build-app
needs:
- changes
uses: ./.github/workflows/app_build.yaml
with:
environment_name: test
@@ -33,7 +44,9 @@ jobs:
deploy-app:
name: deploy-app
needs: build-app
needs:
- changes
- build-app
uses: ./.github/workflows/deploy.yaml
with:
app_name: flying-app

View File

@@ -4,8 +4,14 @@ on:
pull_request:
jobs:
changes:
uses: ./.github/workflows/changes.yaml
build-api:
if: ${{ needs.changes.outputs.api == 'true' }}
name: build-api
needs:
- changes
uses: ./.github/workflows/api_build.yaml
with:
environment_name: pull_request
@@ -13,7 +19,10 @@ jobs:
secrets: inherit
build-app:
if: ${{ needs.changes.outputs.app == 'true' }}
name: build-app
needs:
- changes
uses: ./.github/workflows/app_build.yaml
with:
environment_name: pull_request

View File

@@ -10,7 +10,6 @@ export class AzureAdStrategy extends PassportStrategy(
'azure-ad'
) {
constructor(@Inject(MODULE_OPTIONS_TOKEN) authModuleOptions: AuthModuleOptions) {
console.log(authModuleOptions)
super({
identityMetadata: `https://login.microsoftonline.com/${authModuleOptions.tenantId}/.well-known/openid-configuration`,
clientID: authModuleOptions.clientId,

View File

@@ -118,7 +118,7 @@ const LogForm: React.FC<ILogFormProps> = ({
? { headers: { Authorization: await getAccessToken() } }
: {};
const response: AxiosResponse = await httpClient.get(
`api/logs/${entryId}`,
`api/logs/log/${entryId}`,
config
);
const entry = response.data;

View File

@@ -67,7 +67,6 @@ const SiteNav: React.FC<unknown> = () => {
}
};
const getUserPhoto = async (accessToken: string): Promise<string> => {
console.log(accessToken)
try {
const response: AxiosResponse = await httpClient.get(`api/userPhoto`, {
headers: {