Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

Issue 64 #65

Merged
merged 4 commits into from Oct 19, 2020
Merged

Issue 64 #65

merged 4 commits into from Oct 19, 2020

Conversation

naomijub
Copy link
Owner

@naomijub naomijub commented Oct 18, 2020 •
edited

  • Fixed error. handling when entity functions do not return 2XX.
  • Upgrade edn-rs
  • Removes unnecessary check if id starts with ":"
  • Fix error handling for status code != 2xx in other functions

REMOVED:

if !crux_id.starts_with(":") {
     return Ok(edn!({:status ":bad-request", :message "ID required", :code 400}));
}

@naomijub
Merge pull request #57 from naomijub/beta
d5c1def 
[2.0.0] Enforces types for all functions
@naomijub
Copy link
Owner Author

naomijub commented Oct 18, 2020 •
edited

Resolves issues #64

@naomijub naomijub mentioned this pull request Oct 18, 2020
Closed
evaporei
evaporei reviewed Oct 18, 2020
View reviewed changes
src/http.rs Outdated Show resolved Hide resolved
evaporei
evaporei reviewed Oct 18, 2020
View reviewed changes
src/http.rs Outdated Show resolved Hide resolved
evaporei
evaporei reviewed Oct 18, 2020
View reviewed changes
src/http.rs Show resolved Hide resolved
evaporei
evaporei reviewed Oct 18, 2020
View reviewed changes
src/http.rs Outdated Show resolved Hide resolved
@naomijub naomijub merged commit c30e88b into master Oct 19, 2020
@naomijub naomijub deleted the issue-64 branch October 19, 2020 17:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@evaporei evaporei evaporei approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@naomijub @evaporei