-
Notifications
You must be signed in to change notification settings - Fork 4
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
App Overhaul & Stabilization #175
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…moving .next from gitignore
… to more efficiently set up continuous deployment. applying zip -9 compression to reduce the upload/download time as much as possible.
…emoving that label and trying again.
…esn't seem to be working. reverting back to just creating a .env and populating.
…g over the zip file AND the unzipped contents.
…- maybe this'll help?
… application and started again)
…e by retrieving full coremeasurement row inserted (to get plot/quadrat/personnel info) instead of using user-provided data. Because some of the validations use plot/quadrat, etc., when those validations fail the user-provided data will not provide info about it.
…ll be outlined in PR
…stem that centralizes core function. Interaction testing passed attributes and personnel datagrids
… supporting grids except for species, roles, and quadratpersonnel, none of which are immediately vital.
…ata fields' manual entry
…med and baseline validations have been confirmed to work. Additional stress testing is still needed, but for the time being it should work for piloting.
… fully into a data view.
…ifferent lines and keeping the sidebar from occupying half the screen
siddheshraze
temporarily deployed
to
development
September 16, 2024 19:28 — with
GitHub Actions
Inactive
siddheshraze
temporarily deployed
to
development
September 16, 2024 19:31 — with
GitHub Actions
Inactive
siddheshraze
temporarily deployed
to
development
September 16, 2024 19:59 — with
GitHub Actions
Inactive
siddheshraze
changed the title
Acacia Update -- Full Overhaul to Application
Acacia Update -- App Overhaul & Stabilization
Sep 16, 2024
siddheshraze
temporarily deployed
to
development
September 16, 2024 20:09 — with
GitHub Actions
Inactive
…sions access. shifted the email verification query into the authentication function itself. migration scripts adjusted and updated to correctly work with database. JWT interface extended in place of the Token in the next-auth.d.ts
…t for new table changes. 2. postvalidation API endpoint refined into dynamic routing collection. 3. dedicated page to show post-validation statistics added to measurements hub 4. postvalidationqueries table created in schema. schema structure updated to reflect this 5. layout.tsx file updated to increase debounce (race conditions seem to be occurring). useEffects loop were restructured to ensure that on login, sites will correctly load. 6. error fallback pages received minor edits/changes
… has been retained for application to other places in future.
…dding rds/result mappers for staging/draft tables
siddheshraze
changed the title
Acacia Update -- App Overhaul & Stabilization
App Overhaul & Stabilization
Sep 26, 2024
siddheshraze
temporarily deployed
to
development
September 26, 2024 20:12 — with
GitHub Actions
Inactive
siddheshraze
temporarily deployed
to
development
September 26, 2024 20:20 — with
GitHub Actions
Inactive
justinpac
approved these changes
Sep 26, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Squashing commits to sit here, and also including specific Markdown-formatted overview of changes, this has been pulled from the updated changelog.md file:
App Changelog History
Update (Completion: 9/15/2024)
Note: all project files received updated formatting and linting. Files that received ONLY these changes have been omitted for brevity.
Workflow
Documentation
at this page)
Formatting and Configuration
Testing
Webpages
breakages don't immediately cause cascading issues or full app shutdown.
Dashboard
after site, plot, and census is applied to reduce number of queries being made to schema.
Hub Layout file
selection
externally monitored and only fired when necessary
attempting to re-select site/plot/census. System replaced with staggered loading system and sequential list loading
instead
the site
View Data page
Validations page
etc.
View Full Table page
Sidebar
API changes
frontend/app/api/auth/[[...nextauth]]/route.ts
frontend/app/api/changelog/overview/[changelogType]/[[...options]]/route.ts
frontend/app/api/cmprevalidation/[dataType]/[[...slugs]]/route.ts
frontend/app/api/details/cmid/route.ts
frontend/app/api/fetchall/[[...slugs]]/route.ts
frontend/app/api/fixeddata/[dataType]/[[...slugs]]/route.ts
GET
personnelrole
sitespecificvalidations
roles
measurementssummary
viewfulltable
POST
view
query config implementation updatedPATCH
DELETE
frontend/app/api/postvalidation/route.ts
number of records by quadrat
all stem records by quadrat (count only)
live stem records by quadrat (count only)
tree records by quadrat (count only)
number of dead or missing stems by census
trees outside of plot limits
stems with largest DBH/HOM measurements by species
all trees that were recorded in last census that are NOT in current census
number of new stems per quadrat per census
quadrats with most and least new stems per census
number of dead stems per quadrat per census
number of dead stems per species per census
frontend/app/api/refreshviews/[view]/[schema]/route.ts
frontend/app/api/rollover/[dataType]/[[...slugs]]/route.ts
frontend/app/api/specieslimits/[speciesID]/route.ts
frontend/app/api/sqlmonitor/route.ts
frontend/app/api/structure/[schema]/route.ts
frontend/app/api/validations/crud/route.ts
frontend/app/api/validations/procedures/[validationType]/route.ts
frontend/app/api/validations/updatepassedvalidations/route.ts
frontend/app/api/validations/validationerrordisplay/route.ts
viewed rows with validation errors if they exist
frontend/app/api/validations/validationlist/route.ts
Context Updates
Lock Animation Context
Components Updates
DataGrid Columns
defined in the datagrid components themselves.
alltaxonomiesview
-- specieslimits column customized additionGitHub Feedback Modal
Rollover Modal
Rollover Stems Modal
Validation Modal
event of data loss or some other connection failure event)
DataGrid Updates
isolates as much information as possible to the generic instance rather than the existing DataGridCommons, which
requires parameter drilling of all MUI X DataGrid parameters. Current datagrids using this new implementation are:
alltaxonomiesview
attributes
personnel
quadratpersonnel
quadrats
roles
stemtaxonomiesview
causing cascading failures. Due to the way that runtime utility functions work, no data was actually reaching the
datagrids importing those initial states
the
datagridcolumns.tsx
filemeasurementssummaryview
datagrid instance added as a replacement to the previously defined summary pageRe-Entry Data Modal
key reference columns from reentry)
clusters
andhiddenColumns
added -- theclusters
object provides grouping options for reentry (preventingreentry fields from rendering in a single very very long line) and the
hiddenColumns
object allows columns thatshouldn't need re-entry to
be omitted.
ProcessPersonnel Handler
roles
table has been created andCensusID
andRoleID
foreign keys have been added to the
personnel
tablepersonnel
table nextTheme Updates
100ms and pointerEvents has been set to
none
Upload System Updates
breakdown has also been simplified, but needs updating
to if desired
of all validation procedures
stored as dedicated queries in the
catalog.validationprocedures
table. When validations are run, the respective SQLqueries is pulled from the table. This makes the system flexible and modifiable, as stored procedures are
significantly more immutable.
measurementssummary
andviewfulltable
. Due to extended query time for the existingSQL views and the lack of support for materialized views in MySQL 8, a manual version was implemented. these
dedicated tables are populated via stored procedure and have significantly reduced query time. Upload system has been
enhanced to call both table refresh commands as part of the Azure upload process.
more of the views fails.
SQL Updates
roles
- outlines user rolesspecieslimits
- allows setting min/max bounds on measurementsspecimens
- recording specimen data (added on request by ForestGEO)unifiedchangelog
- partitioned table that tracks all changes to all tables in schema. All tables have triggersthat automatically update the
unifiedchangelog
on every changesitespecificvalidations
- for specific validations applicable only to the host sitevalidationprocedures
andsitespecificvalidations
tables