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.

Sign up for GitHub

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

Jump to bottom

[GWELLS-2011/2013] FEATURE** New attachments UI #2035

Merged
merged 28 commits into from
Nov 7, 2023
Merged

[GWELLS-2011/2013] FEATURE** New attachments UI #2035

merged 28 commits into from
Nov 7, 2023

Conversation

LocalNewsTV
Copy link
Contributor

Pull Request Standards

  • The title of the PR is accurate
  • The title includes the type of change [HOTFIX, FEATURE, etc]
  • The PR title includes the ticket number in format of [GWELLS-###]
  • Documentation is updated to reflect change

Description

This PR includes the following proposed change(s):

  • Updated Bootstrap library
  • Streamlined way to upload files using a small form, removing guesswork and human error
  • Ability to add date of action to the filename, showing more meaningful information at a quick glance
  • New way to visualize attachments related to a well, based on the file name being uploaded

Table displaying data

image

Uploading form

image

davidclaveau and others added 21 commits October 26, 2023 17:00
@davidclaveau
wip - rework attachments table
1e9f2ef
@davidclaveau
add document label codes
a3eb77c
@davidclaveau
wip - refactor Documents component
d9af5ee
@LocalNewsTV
automate file name in uploader
4dfcf3a
@LocalNewsTV
networked docker containers
e4d62cf
@LocalNewsTV
change local API target from backend -> localhost
9a5f7c0
@LocalNewsTV
locked uploadable file types
2eea134
@LocalNewsTV
update bootstrap vue package
bdf62f1
@LocalNewsTV
Create object maps for Well labels
82862d6
@LocalNewsTV
Added file_name logic and datepicker
2738199
@LocalNewsTV
update table headers for updated bootstrap package
54812e2
@LocalNewsTV
update table header objects
e979c9d
@LocalNewsTV
update API to clean up returned public objects
6ee7ba9
@LocalNewsTV
add longform labels to table
7a29b2a
@davidclaveau
wip - fix to upload docs from edit well
164a32d
@LocalNewsTV
Added table to edit docs, removed old ul's
c31edb6
@davidclaveau
fix to doc upload, allow deletion from edit
368f6be
@LocalNewsTV
code cleanup, change header names for clarity, fix public list bug
12a1857
@LocalNewsTV
bug fix, documented getLongFormLabel
3b33b57
@LocalNewsTV
Files uploaded now flagged correctly (private)
9a4e724
@LocalNewsTV
documentation added
d47b16b
@LocalNewsTV LocalNewsTV changed the title [NRPTI-2011/2013] FEATURE** New attachments UI [GWELLS-2011/2013] FEATURE** New attachments UI Nov 6, 2023
@LocalNewsTV LocalNewsTV closed this Nov 7, 2023
@LocalNewsTV LocalNewsTV reopened this Nov 7, 2023
@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Nov 7, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@LocalNewsTV LocalNewsTV merged commit 50525e7 into release Nov 7, 2023
11 checks passed
@LocalNewsTV LocalNewsTV deleted the feature/documents-storage-overhaul-2 branch November 7, 2023 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidclaveau davidclaveau davidclaveau approved these changes

@sanjaytkbabu sanjaytkbabu sanjaytkbabu approved these changes

Assignees
No one assigned
Labels
Backend Docker Frontend OpenShift
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@LocalNewsTV @davidclaveau @sanjaytkbabu @bcgov-csnr-cd