[PR #217] [CLOSED] [Auth restructure] Making sure both PRs are the same #321

New issue

Closed

opened 2026-02-27 20:24:15 +03:00 by kerem · 0 comments

kerem commented

2026-02-27 20:24:15 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ramsayleung/rspotify/pull/217
Author: @marioortizmanero
Created: 6/19/2021
Status: ❌ Closed

Base: auth-rewrite-part4 ← Head: auth-rewrite-part5

📝 Commits (10+)

05f906b moving http to a new crate
68d625f rspotify-http now working more or less
bbbb3af moving endpoints to respective traits
1a4487b hit a wall with impl
61edf5d it compiles... but at what cost
82f1825 format
b47f273 numerous fixes
1107624 some fixes
3a0bb68 Merge with master
02c4c0c format

📊 Changes

46 files changed (+3877 additions, -3977 deletions)

View changed files

📝 .github/workflows/ci.yml (+7 -29)
➖ .travis.yml (+0 -25)
📝 Cargo.toml (+27 -36)
➕ examples/auth_code.rs (+55 -0)
➕ examples/auth_code_pkce.rs (+47 -0)
📝 examples/client_creds.rs (+10 -14)
➖ examples/current_user_recently_played.rs (+0 -51)
📝 examples/oauth_tokens.rs (+7 -12)
📝 examples/pagination_async.rs (+6 -36)
📝 examples/pagination_manual.rs (+6 -36)
📝 examples/pagination_sync.rs (+6 -36)
➖ examples/track.rs (+0 -42)
➖ examples/tracks.rs (+0 -42)
📝 examples/ureq/device.rs (+6 -36)
📝 examples/ureq/me.rs (+6 -36)
📝 examples/ureq/search.rs (+17 -47)
📝 examples/ureq/seek_track.rs (+6 -36)
📝 examples/webapp/src/main.rs (+71 -78)
📝 examples/with_refresh_token.rs (+11 -26)
➕ rspotify-http/Cargo.toml (+54 -0)

...and 26 more files

📄 Description

This makes sure the split up is the same as the original PR, I'll remove it later.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ramsayleung/rspotify/pull/217 **Author:** [@marioortizmanero](https://github.com/marioortizmanero) **Created:** 6/19/2021 **Status:** ❌ Closed **Base:** `auth-rewrite-part4` ← **Head:** `auth-rewrite-part5` --- ### 📝 Commits (10+) - [`05f906b`](https://github.com/ramsayleung/rspotify/commit/05f906bc1092b8105047ce7ecbbf7062dfdc18d7) moving http to a new crate - [`68d625f`](https://github.com/ramsayleung/rspotify/commit/68d625f584069f34f8016324adf4c036a0a556f9) rspotify-http now working more or less - [`bbbb3af`](https://github.com/ramsayleung/rspotify/commit/bbbb3af82ce1cc7a0c79f7c15d4decd978f287e1) moving endpoints to respective traits - [`1a4487b`](https://github.com/ramsayleung/rspotify/commit/1a4487b121922f7014e7d11e8f117c3f6aaa1422) hit a wall with `impl` - [`61edf5d`](https://github.com/ramsayleung/rspotify/commit/61edf5dbb6acfdf42b01dd7b8369a0311a051d4b) it compiles... but at what cost - [`82f1825`](https://github.com/ramsayleung/rspotify/commit/82f1825ed87c3c88bac0d164aac552c2ad1616ae) format - [`b47f273`](https://github.com/ramsayleung/rspotify/commit/b47f27379025867084c49d1cf3286c4bdd8e8430) numerous fixes - [`1107624`](https://github.com/ramsayleung/rspotify/commit/1107624f380de0a8ef573f701e6b4aa44224ac15) some fixes - [`3a0bb68`](https://github.com/ramsayleung/rspotify/commit/3a0bb68db68477278a751516292e577692c14fd9) Merge with master - [`02c4c0c`](https://github.com/ramsayleung/rspotify/commit/02c4c0ce571da5184ad00333d632086289f46f0b) format ### 📊 Changes **46 files changed** (+3877 additions, -3977 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+7 -29) ➖ `.travis.yml` (+0 -25) 📝 `Cargo.toml` (+27 -36) ➕ `examples/auth_code.rs` (+55 -0) ➕ `examples/auth_code_pkce.rs` (+47 -0) 📝 `examples/client_creds.rs` (+10 -14) ➖ `examples/current_user_recently_played.rs` (+0 -51) 📝 `examples/oauth_tokens.rs` (+7 -12) 📝 `examples/pagination_async.rs` (+6 -36) 📝 `examples/pagination_manual.rs` (+6 -36) 📝 `examples/pagination_sync.rs` (+6 -36) ➖ `examples/track.rs` (+0 -42) ➖ `examples/tracks.rs` (+0 -42) 📝 `examples/ureq/device.rs` (+6 -36) 📝 `examples/ureq/me.rs` (+6 -36) 📝 `examples/ureq/search.rs` (+17 -47) 📝 `examples/ureq/seek_track.rs` (+6 -36) 📝 `examples/webapp/src/main.rs` (+71 -78) 📝 `examples/with_refresh_token.rs` (+11 -26) ➕ `rspotify-http/Cargo.toml` (+54 -0) _...and 26 more files_ </details> ### 📄 Description This makes sure the split up is the same as the original PR, I'll remove it later. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>