Then the instance of the React class has access to it at this.props.revision
CatQuest
woa i let the archive.org and even the error message is now gone
<-- idiot
LordSputnik
stanislas: then you can access the properties of this.props.revision inside the React element, and pull out whatever you need
stanislas
Yeah, I understand that, thanks. What about the question regrarding if ?
Freso
gcilou: "AcousticBrainz is a joint effort between Music Technology Group at Universitat Pompeu Fabra in Barcelona and the MusicBrainz project." → "AB is a joint effort with MTG at UPF in Barcelona." (with "MTG" and "UPF" being links)
gcilou
same with IA?
Freso
"Cover Art Archive is a joint project with the Internet Archive." (with t... yes. Same thing. :))
LordSputnik
stanislas: Ahhh I see what you mean now
gcilou
:P
Freso
gcilou: The page is already specifying that they're MetaBrainz projects, so there's no reason to re-state that.
gcilou
okay
where?
LordSputnik
stanislas: define a variable to hold the content, then assign it with different content in different branches of the if statement. Then put the variable inside the returned react element in the class
("MusicBrainz" is used on those pages because 1) MB is a far more well-known "brand" than MeB is, and 2) CAA and AB relate directly to MB data, not to data from other MeB projects (though I guess AB is also using MsB now/soon).)
LordSputnik
(here, loadingSpinner is using the conditional operator, but same concept)
Freso
gcilou: "The MetaBrainz Foundation Projects"?
gcilou
oh
LordSputnik
stanislas: that OK?
gcilou
so just "The Projects"?
stanislas
Yeah, I got it.
LordSputnik: I thought that maybe there is some way like in jijna2
Freso
gcilou: No, I think the header is fine.
stanislas
LordSputnik: thanks :)
gcilou
oh okay I was confused but now I got it
Freso
gcilou: But there's no reason to report for project that they're MetaBrainz projects, as that's the whole premise of that page.
*repeat
gcilou
ok
Freso
If it wasn't a MetaBrainz project, it shouldn't be listed on that page to begin with. :)
ariscop joined the channel
LordSputnik
stanislas: no, because React focuses on using HTML within code, rather than code within HTML. It's more about building up/composing the HTML and defining new elements than templating stuff
stanislas: That's one of the key differences between React and templating languages, and something which I hope will benefit us in the long run
stanislas
LordSputnik: Seems clear to me. Thanks for explaining the difference.
I am curious how come this task is easier than 'create one entity on MB' task :)
Freso
gcilou: Does my reasoning make sense to you? :)
(It doesn't always to me, so it'd be fair.)
gcilou
Yes yes it does.
LordSputnik
stanislas: it's not easier, but it's beginner because it introduces the student to the bookbrainz-site codebase in a fairly non-scary way :)
gcilou
Freso, should be better now
LordSputnik
eg. you don't have to worry about setting up WS interaction, showing error messages or configuring the build system :D
stanislas
The file you've linked in is great. I love to learn by examples if it possible and this one include nearly all of the stuff I want.
Freso
gcilou: Looks better, yep. :)
gcilou
yay
Freso
I think when Gentlecat is happy, I'm happy.
gcilou
Good. Is Gentlecat happy?
Freso
Oh, no!
D:
gcilou
??
Freso
You've HTTPS'd the AB logo link.
AB doesn't work over HTTPS at this point.
So fix that, and *then* I'm happy. ;)
gcilou
urg. that weird why did I do that.
Freso
Also not sure whether to link MsB as HTTPS or HTTP.
HTTPS *does* work, but the cert. is broken.
gcilou
so take it out for now?
Freso
And I'm not sure if it's going to be fixed up.
zas: ^
Yeah.
It can be added in whenever we set up HTTPS for MsB properly.
(Automatically regenerating Let's Encrypt certificates or however.)
Freso goes to make a MsB PR...
gcilou
Freso, fixed
reosarevok
Freso: is there any link to the MeB YT channel on the site? if not, where should we put it?
Freso
No clue. :p
reosarevok
Also, any idea when you'll publish that relationship editor video? I was going to link a new user to it but I see it's not out yet :)
Re: "for some projects you say "<project> is a...", for others "<project> <is trying to do something>"" - I think it's fine to both leave as is, but it'd also be fine to rewrite so they all sound similar.