Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compilation warnings and other warnings in albs-frontend #434

Closed
javihernandez opened this issue Feb 14, 2025 · 0 comments · Fixed by AlmaLinux/albs-frontend#589
Closed
Assignees
Labels
ui UI (frontend) related issues

Comments

@javihernandez
Copy link
Member

We can see the following compilation warnings:

frontend-1  |  App •  WARNING  •  UI  in ./src/pages/TestLogs.vue?vue&type=template&id=2894e585
frontend-1  | 
frontend-1  | Module Warning (from ./node_modules/@quasar/app-webpack/node_modules/vue-loader/dist/templateLoader.js):
frontend-1  | 
frontend-1  | <tr> cannot be child of <table>, according to HTML specifications. This can cause hydration errors or potentially disrupt future functionality.
frontend-1  | 2  |    <div class="row no-wrap no-scroll q-pa-md">
frontend-1  | 3  |      <table class="q-table horizontal-separator full-width col-lg-12 col-xl-10">
frontend-1  |  App •  WARNING  • Compilation succeeded but there are warning(s). Please check the log above.
frontend-1  | 4  |        <tr>
frontend-1  | 
frontend-1  |    |        ^^^^
frontend-1  | 5  |          <td colspan="2">
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 6  |            <q-input placeholder="Search" v-model="search" debounce="300">
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 7  |              <template v-slot:append>
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 8  |                <q-icon name="search" />
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 9  |              </template>
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 10 |            </q-input>
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^
frontend-1  | 11 |          </td>
frontend-1  |    |  ^^^^^^^^^^^^^
frontend-1  | 12 |          <td>
frontend-1  |    |  ^^^^^^^^^^^^
frontend-1  | 13 |            <q-select
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^
frontend-1  | 14 |              v-model="statusFilter"
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 15 |              label="Status filter"
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 16 |              radio
frontend-1  |    |  ^^^^^^^^^^^^^^^^^
frontend-1  | 17 |              :options="testsStatusFilterLabels"
frontend-1  |    |  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
frontend-1  | 18 |            />
frontend-1  |    |  ^^^^^^^^^^^^
frontend-1  | 19 |          </td>
frontend-1  |    |  ^^^^^^^^^^^^^
frontend-1  | 20 |        </tr>
frontend-1  |    |  ^^^^^^^^^^^

In addition to this, while loading the ALBS, I can see the following warning in console:

Image

@javihernandez javihernandez added the ui UI (frontend) related issues label Feb 14, 2025
@javihernandez javihernandez self-assigned this Feb 14, 2025
@javihernandez javihernandez moved this to In progress in AlmaLinux Build System Feb 14, 2025
javihernandez added a commit to javihernandez/albs-frontend that referenced this issue Feb 14, 2025
Also, add id property to EssentialLinks component to fix navigation
warnings about non-props attributes.

Fixes: AlmaLinux/build-system#434
javihernandez added a commit to javihernandez/albs-frontend that referenced this issue Feb 14, 2025
Also, add id property to EssentialLinks component to fix navigation
warnings about non-props attributes.

Fixes: AlmaLinux/build-system#434
@javihernandez javihernandez moved this from In progress to In review in AlmaLinux Build System Feb 14, 2025
javihernandez added a commit to AlmaLinux/albs-frontend that referenced this issue Feb 18, 2025
Also, add id property to EssentialLinks component to fix navigation
warnings about non-props attributes.

Fixes: AlmaLinux/build-system#434
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ui UI (frontend) related issues
Projects
Status: In review
Development

Successfully merging a pull request may close this issue.

1 participant