Skip to content

Remove old DNS TXT schema#2139

Closed
Kubuxu wants to merge 2 commits intoipfs:dev0.4.0from
Kubuxu:fix/#760
Closed

Remove old DNS TXT schema#2139
Kubuxu wants to merge 2 commits intoipfs:dev0.4.0from
Kubuxu:fix/#760

Conversation

@Kubuxu
Copy link
Copy Markdown
Member

@Kubuxu Kubuxu commented Dec 29, 2015

As for now only dnslink is valid.
Fixes #760

License: MIT
Signed-off-by: Jakub (Kubuxu) Sztandera kubuxu@gmail.com

@jbenet jbenet added the backlog label Dec 29, 2015
@Kubuxu
Copy link
Copy Markdown
Member Author

Kubuxu commented Dec 30, 2015

Should I recreate this PR for dev0.4.0?

@ghost
Copy link
Copy Markdown

ghost commented Dec 30, 2015

Should I recreate this PR for dev0.4.0?

Looks like it's already based off dev0.4.0 right?

@Kubuxu
Copy link
Copy Markdown
Member Author

Kubuxu commented Dec 30, 2015

I wrote under wrong PR. Sorry for that.

(I should have been sleeping instead of codding and looking through github).

@daviddias daviddias removed the backlog label Jan 2, 2016
@ghost ghost added topic/ipns Topic ipns topic/cleanup Topic cleanup RFCR labels Jan 2, 2016
@ghost ghost changed the title Remove old DND TXT schema Remove old DNS TXT schema Jan 2, 2016
whyrusleeping and others added 2 commits January 12, 2016 18:02
As for now only dnslink is valid.
Fixes ipfs#760

License: MIT
Signed-off-by: Jakub (Kubuxu) Sztandera <kubuxu@gmail.com>
@Kubuxu
Copy link
Copy Markdown
Member Author

Kubuxu commented Jan 12, 2016

It is set to merge to dev0.4.0. Instead of merging it via GH, you have to manually pull this branch into master.

@jbenet
Copy link
Copy Markdown
Member

jbenet commented Jan 13, 2016

@Kubuxu will be easier if you push a new PR i think. we want CI to run on top of master.

@Kubuxu
Copy link
Copy Markdown
Member Author

Kubuxu commented Jan 13, 2016

See #2193.

@Kubuxu Kubuxu closed this Jan 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic/cleanup Topic cleanup topic/ipns Topic ipns

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants