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
ForestGEO version 2.1.0 -- Fixing bugs and upgrading core functionality #165
Open
siddheshraze
wants to merge
162
commits into
main
Choose a base branch
from
feature-development
base: main
Could not load branches
Branch not found: {{ refName }}
Could not load tags
Nothing to show
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+5,603
−2,088
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
… application --> will make sure that modal does not redisplay while navigating around hub routes
…be somewhat dynamic, and not all endpoints require quadrat/census selection, the previous system of only plot selection will be fitted in place fo the plot/quadrat/census breadcrumb system that was previously in place
…g quadratsperplot API endpoint
…access to other contexts
…drat lists and census list
…drat lists and census list
…drat lists and census list
…ented. Spacing added in files to better show function
divided uploaded file display into two displays, one for CSV and one for ArcGIS , additional changes to API structure to accommodate new schema and refit system to start accepting file uploads to specific tables instead of just single-type file uploads
…to add prompt to identify which table is being uploaded to.
…ated --> adding it back
…rror return from api call to storage account
… a user is signing back in or logging in for the first time. resolving additional sonarlint warnings/errors in sidebar, etc. updating schema generation script
…g file upload system to dynamically process files based on detected headers, etc. reworking other elements of the system to utilize MaterialUI instead of TailwindCSS and updating plugins
…s been implemented to partially build out the dynamic file upload system. adding baseline Jest files to start the process of building out tests
… user selection when uploading files
…lection back and referencing the existing plotselection and censusselection systems in entrymodal and endpoint to create a third selection matrix for quadrats for future use. updating the container client creation/selection system to isolate by plot && census instead of just plot to better track fixed background data file input
…to just show current date instead of null or invalid date
…lly disabled pool monitor export so restoring that super quick
…on bugs out the app (census 1 has a null start date)
…o ensure that attributes, which delete via string code values, are correctly accounted for when interacting with the datagrid
…ove unused imports, etc.
…cessed properly from parsing to upload, so adding additional user information panels and custom handling to account for that.
…oved to file upload stage 2. Measurements Summary Grid Columns' measurementDate column updated to type string. Moment-based date check added and conversion to date and then to datestring added. 3. date-fns typescript package added 4. upload parent compartmentalized into new separate modal component. All datagrid views updated to use this in place of existing manual modal creation 5. uploadparentmodal enhanced to restrict users from exiting modal by any means other than pressing the X button. 6. uploadstart stage updated to simplify and reduce complexity. components moved to center and back/finalize button adjusted to be more visually appealing.
…ht active parts of the upload system and separate them from the administrative parent component uploadparent.tsx 2. added modal system to compartmentalize upload parent system into dedicated state-managed variable system
…te and datagridhelpers.ts handling to show validation history (future-proofing, not currently in use)
…anizing formatting, etc
1. datagridcommons (and validation procedures) -- multiple rows with validation errors were not being highlighted (only one at a time). After investigating, it was determined that the bug was in the structuring of the validation procedures, which was overwriting or removing past validation errors for a given coremeasurementID. It seems to be working now. 2. sidebar -- sidebar-independent scrolling implemented to ensure that main page stretching does not occur. 3. login -- automated slideshow of background images was only running once. interval system and reset implemented to ensure that it runs continuously when the login page is open. 4. measurementssummary -- API route updated to remove duplicate validation error querying, which was also causing issues with the row error display. Output was also not being parsed correctly, leading to problems with pagination, now corrected. 5. styleddatagrid -- border outlining was not rendering properly, corrected New features/functionality: 1. stemtreedetails -- page created in measurementshub, custom view and route implemented, and configuration added to generic datagrid. helper functions added as needed, and RDS macros implemented. 2. dynamic routing system implemented for validation procedures -- all manual routes have been collapsed into one route 3. default limits -- species table updated to include default max/min DBH & HOM values that are now queried in place of using hardcoded values. 4. subspecies -- page created in properties hub. Currently unpopulated, but CRUD system will need testing to ensure correct function.
siddheshraze
force-pushed
the
feature-development
branch
from
April 18, 2024 16:00
b800721
to
193ef16
Compare
…ructuring for stemTreeDetails
2. removing azure storage container text 3. removing endless login page slide loop (set to 30s) 4. removing endless icon loop (set to 30s)
…rences added directly to datagridcommons instead of passing it in via parameter.
…ed back now for some reason. removing it again
…without accepting any changes from main
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.
Changelog