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

[WIP] 0.7.0 preview #110

Merged
merged 133 commits into from
Oct 5, 2018
Merged

[WIP] 0.7.0 preview #110

merged 133 commits into from
Oct 5, 2018

Conversation

davidcaron
Copy link
Contributor

@davidcaron davidcaron commented Oct 3, 2018 •
edited
Loading

This PR contains #104 #106 and #109

Current status on colibri:

Waiting for this PR to merge/deploy: https://github.com/Ouranosinc/PAVICS/pull/99

@davidcaron
add remote resources from the groups view
8408d61
@davidcaron
default service_url bug
eb889b5
@davidcaron
groups view: add flag + button to remove a resource not on remote server
446ca7e
@davidcaron
add warning when there was an error fetching the data from remote server
6106f24
@davidcaron
Clarify how to implement a new service by using an abstract class
487de88
@davidcaron
Implement Geoserver
47149b0
@davidcaron
allow integers for resource_names
890f521
@davidcaron
Save remote resources in a separate table.
71c4649 
This is the base for a cron service to be ran periodically.
The local and remote resource tables are compared when the
view is requested.
@davidcaron
Permission checkboxes post a 'permission' value
3b8e1b8 
instead of a 'resource_id'
To distinguish permission requests and 'clean_resource' requests
@davidcaron
Fix post request to edit permissions
4b1df51
@davidcaron
Display resources sorted alphabetically
11755f5
@davidcaron
Cleanup and document
e4cdb3e
@davidcaron
Display resources sorted in edit service view
aeea8aa
@davidcaron
ui color change
fb2c6c2
@davidcaron
last sync info and clean all button
d9b58c7
@davidcaron
also write resource type to database
49ddb13
@davidcaron
rename
dad9f72
@davidcaron
add resource_type attribute to resources
bd08cae
@davidcaron
reformat
0b076b1
@davidcaron
basic housekeeping function
23f03d1
@davidcaron
use service type instead of service name
a7de1da
@davidcaron
show message if sync is not implemented
20bda2c
@davidcaron
implement project-api
52a82b7
@davidcaron
refactoring
94889cd
@davidcaron
services are uniquely identified using both their type and resource_name
3e49695
@davidcaron
fix adding a nested resource
23c52ba
@davidcaron
don't add remote_path if it doesn't match remote resource
f014586
@davidcaron
fix 'Clean' button for a single resource
11c7c16
@davidcaron
fix housekeeping algorithm
e239217
@davidcaron
add logging
00eed72
@fmigneault
Copy link
Collaborator

500 is internal error, so project-api as an error when executing GET Projects, but service is up otherwise.
@Renaud009 you know about this issue?

@fmigneault
Copy link
Collaborator

Using latest magpie, I got an error related to new version of gunicorn.
Using following patch resolved the issue:

Magpie/requirements.txt

Line 24 in 7989367

gunicorn==19.8.1

We should cherry-pick this tweak into 0.7.0 version

@davidcaron
Copy link
Contributor Author

@fmigneault is there a reason logging was added in requirements.txt? It got removed by cherry-picking for the gunicorn version.

@fmigneault
Copy link
Collaborator

Removed as it's not necessary, was getting an error because of pycharm acting up, but no code issue.

@fmigneault fmigneault mentioned this pull request Oct 5, 2018
Merged
@fmigneault fmigneault merged commit 034ad92 into master Oct 5, 2018
This was referenced Oct 5, 2018
Closed
Closed
@fmigneault fmigneault deleted the 0.7.0-preview branch December 6, 2018 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@davidcaron @fmigneault