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

[MDS-6024] All Documents Tab #3195

Merged
merged 16 commits into from
Jul 26, 2024
Merged

[MDS-6024] All Documents Tab #3195

merged 16 commits into from
Jul 26, 2024

Conversation

taraepp
Copy link
Collaborator

@taraepp taraepp commented Jul 25, 2024

Objective

  • add new content, layout, to all documents tab
  • fix IRT nav on CORE
  • is it loading? Make the loading bar more visible, and the spinner more consistent
  • scroll side page wrapper can be used in different ways now

TO DO:

  • Minespace styling! Horizontal alignment in layout is bad (it overlaps, and also the content extends beyond the right page boundary)
  • double check heading sizes, styling, spacing (it's not perfect but added Row/Col/gutter, Alert, and fixed heading sizes)
  • would be nice if the random looking "top" values in stylesheet were at least composed of variables and didn't look completely arbitrary (& difficult to maintain if padding/margin changes)
  • another ticket: make sure that archived AMS authorization documents come back from the document fetch and then turn on archiving
  • the buttons in the bulk "Actions" dropdown look really weirdly styled
  • implement the "Download All" on CORE for all the documents on the page (I think should be fairly simple?)
  • and maybe the Actions (for all of the documents) if it's doable

MDS-6024

Why are you making this change? Provide a short explanation and/or screenshots

@taraepp taraepp marked this pull request as ready for review July 25, 2024 21:28
import { Loading as CommonLoading } from "@mds/common/components/common/Loading";

const Loading = () => {
return <CommonLoading />;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice to see load spinner be more consistent across minespace and core

simensma-fresh
simensma-fresh previously approved these changes Jul 26, 2024
<div id={href}>
<Typography.Title level={titleLevel}>
<DeleteOutlined
// className="violet"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can probably remove this

# Conflicts:
#	services/common/src/interfaces/projects/projectSummary.interface.ts
Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_common'

Failed conditions
78.0% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_minespace-web'

Failed conditions
41.3% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

Copy link

Quality Gate Failed Quality Gate failed for 'bcgov-sonarcloud_mds_core-web'

Failed conditions
47.5% Coverage on New Code (required ≥ 80%)

See analysis details on SonarCloud

@henryoforeh-dev henryoforeh-dev merged commit 537f1f6 into develop Jul 26, 2024
12 of 15 checks passed
@henryoforeh-dev henryoforeh-dev deleted the mds-6024-all-docs branch July 26, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants