Closed
Conversation
added 10 commits
September 18, 2020 09:18
- add MobilityData's logo - add input field - add validation button - install semantic-react-ui dependency
- make input field width larger - add licence header
|
Cool, looking promising. As discussed, next steps before possibly merging to master
|
This was referenced Sep 21, 2020
Closed
* Add support to allow CORS * Add support to package web app as jar -add dependency on client module - add task to build jar in client module * Adapt GitHub workflow to generate .jar from spring boot application * Touch * Modify default port for server launch * Use relative default path to access validation report * Modify GitHub workflow to run checks on different branch * Reset hard coded path to validation report * Remove support for CORS since irrelevant now * Add configuration file to prevent generation of package-lock.json https://www.codementor.io/@johnkennedy/get-rid-of-that-npm-package-lock-json-e0bj7ai42 * Add unit test to check get mapping in spring boot application * Modify GitHub workflow - persist jar of web app - rework action names * Modify build.gradle files to adapt name of built artifacts * Prep next release * Bump version number * Add details for execution of bootJar command * Update README.md and RELEASE.md with instructions regarding web app * Update link to screenshot in documentation * Implement #382 - Rework UI (#381) * Rename file for clarity * Add dropzone elements into UI * Add support to render execution parameter configuration file within UI after drag and drop action * web client: Add support to review exec param config file * Implement builder pattern for DefaultConfig * Add dependency to beautify json files * Implement validation of GTFS archive * Remove 80% opacity on dropzone * Improve rendering of json content as table * Add support on client side to handle fee validation * Rename files according to conventions * Disable caching * Improve code modularity - create Constants.js file to store constants - create ApiRequest.js file to store functions used to call API * Server: Implement method to get path to report * Add eof line * Server side: add unit tests - test controller - test service manager - add exception handling * remove redundant time consuming test * Implement test of interface with jet * Add dependency on librairies: - axios: http requests: - material-ui: progress circles * - Leverage PostMapping in order to pass data to backend - Rename methods to be consistent with http requests - Add progress circle - Add support to clear HTML on drop of other configuration file * Additional unit tests * Comment test to be fixed * Add explicit call to method deleteTable to avoid no-unused-vars error * Change validation success message for clarity * Center display validation report report * Add console comment to inform success of application start - add link to http://localhost:8090 * Add option --beautify - define said option as "true" by default - add new key to option list - adapt mechanisms to retrieve value - define default value in default-execution-parameters.json - adapt tests - write additional tests to verify said implementation - Remove empty lines * Block interface interactions when validation process is running - add props.id to button for reference - add commands to release/hold buttons and dropzone * Modify GitHub workflow * Remove unused script * Make CircularProgress bigger - keep only 1 CircularProgress - change color to blue (Default) * Leverage JsonPretty to display examples of configuration files * Adjust CSS for table rendering json configuration file * Remove service worker usage * Update icons and page title * Update README.md * Update README.md * Remove unused components * Adjust import statements * Rename components * Rename components * Touch * Rename module client->react client * Rename module client->react client * Reset GitHub workflow * PR feedback - fix typos - improve documentation - update GitHub workflow * - update GitHub workflow
- write new constructor - update tests
# Conflicts: # .github/workflows/gradle.yml # application/cli-app/build.gradle # config/src/main/java/org/mobilitydata/gtfsvalidator/config/DefaultConfig.java
ghost
suggested changes
Oct 6, 2020
ghost
left a comment
There was a problem hiding this comment.
We're almost there, very neat!
A few changes inline
application/cli-app/src/main/java/org/mobilitydata/gtfsvalidator/Main.java
Show resolved
Hide resolved
...pring-boot-app/src/test/java/org/mobilitydata/gtfsvalidator/GtfsValidatorControllerTest.java
Show resolved
Hide resolved
ghost
suggested changes
Oct 6, 2020
ghost
left a comment
There was a problem hiding this comment.
A few changes still needed. See inline
...r/exporter/src/main/java/org/mobilitydata/gtfsvalidator/exporter/ProtobufNoticeExporter.java
Outdated
Show resolved
Hide resolved
...r/exporter/src/main/java/org/mobilitydata/gtfsvalidator/exporter/ProtobufNoticeExporter.java
Outdated
Show resolved
Hide resolved
...porter/src/test/java/org/mobilitydata/gtfsvalidator/exporter/ProtobufNoticeExporterTest.java
Outdated
Show resolved
Hide resolved
...porter/src/test/java/org/mobilitydata/gtfsvalidator/exporter/ProtobufNoticeExporterTest.java
Outdated
Show resolved
Hide resolved
...mple/src/main/java/org/mobilitydata/gtfsvalidator/db/InMemoryValidationResultRepository.java
Outdated
Show resolved
Hide resolved
added 9 commits
October 7, 2020 09:20
Contributor
Author
|
This PR will be closed without merge. To follow-up the work done in, please refer to #421. |
|
Content was migrated and merged through #421 |
This pull request was closed.
This file contains hidden or 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
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.
The content of this PR was migrated to #421
Summary:
This PR provides support to develop a local web ui based on the existing gtfs validator using spring boot, and react/
Expected behavior:
./gradlew bootRun-> will start by default on server port 8080,To run the web app :
yarn startPlease make sure these boxes are checked before submitting your pull request - thanks!
gradle testto make sure you didn't break anything